Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[KemonoBridge] Add KemonoBridge #4192

Merged
merged 3 commits into from
Aug 10, 2024
Merged

[KemonoBridge] Add KemonoBridge #4192

merged 3 commits into from
Aug 10, 2024

Conversation

Phantop
Copy link
Contributor

@Phantop Phantop commented Aug 5, 2024

Adds bridge for https://kemono.su, pulling in textual content, attachments, external embeds, and tags for all posts for a given service and user.

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 5, 2024 •
edited
Loading

Pull request artifacts

Bridge Context Status
Kemono 1 untitled (pr) ✔️

last change: Friday 2024-08-09 04:37:08

@dvikan
Copy link
Contributor

dvikan commented Aug 7, 2024 

504 - Gateway Timeout . That’s an error.

@Phantop
Copy link
Contributor Author

Phantop commented Aug 8, 2024 •
edited
Loading 

504 - Gateway Timeout . That’s an error.

Where are you seeing this issue? A bad gateway error is usually a temporary thing so I don't think it's necessarily an issue with the bridge. I have a live version of it running on my own rss-bridge instance here.

@dvikan
Copy link
Contributor

dvikan commented Aug 8, 2024

both digital ocean and my home isp (Norway)

@Phantop
Copy link
Contributor Author

Phantop commented Aug 8, 2024

both digital ocean and my home isp (Norway)

Can I have the full error? Do you have a line of code responsible? Can you access https://kemono.su normally?

@dvikan
Copy link
Contributor

dvikan commented Aug 8, 2024

im unable to access https://kemono.su in regular browser firefox.

504 - Gateway Timeout . That’s an error.

We did not receive a timely response from the upstream server. That’s all we know.

host kemono.su
kemono.su has address 190.115.16.14
kemono.su mail is handled by 10 mail.kemono.ru.

$ curl -i 'https://kemono.su/'
HTTP/2 504 
server: ddos-guard
date: Thu, 08 Aug 2024 01:38:00 GMT
content-type: text/html; charset=utf-8
content-length: 583

<!DOCTYPE html><html lang=en><meta charset=utf-8><meta name=viewport content="initial-scale=1, minimum-scale=1, width=device-width"><title>Error 504</title><style>*{margin:0;padding:0}html{font:15px/22px arial,sans-serif;background: #fff;color:#222;padding:15px}body{margin:7% auto 0;max-width:390px;min-height:180px;padding:30px 0 15px}p{margin:11px 0 22px;overflow :hidden}ins{color:#777;text-decoration :none;}</style><p><b>504 - Gateway Timeout .</b> <ins>That’s an error.</ins><p>We did not receive a timely response from the upstream server.  <ins>That’s all we know.</ins>

@Phantop
Copy link
Contributor Author

Phantop commented Aug 8, 2024

Hmm that must be in some form related to your ISP or region (and, somehow, DigitalOceans?). You can see the site is valid based on the output of the PR artifact action and the archive of the homepage is just made here.

@Phantop
Copy link
Contributor Author

Phantop commented Aug 10, 2024

Any update on this? The 504 issue is unrelated to the bridge itself, so I don't think there's anything that can be done about it, unfortunately.

@dvikan dvikan merged commit 4ef5ca5 into RSS-Bridge:master Aug 10, 2024
7 checks passed
@dvikan
Copy link
Contributor

dvikan commented Aug 10, 2024

i think there is a redundant call to getContents. check it out pls

@Phantop
Copy link
Contributor Author

Phantop commented Aug 10, 2024

i think there is a redundant call to getContents. check it out pls

Unfortunately, it is required as the API endpoint with the entries does not provide the feed title, which is what the second API call is for.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Phantop @dvikan