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.

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

[ExplosmBridge] Call to a member function find() on null #2415

Closed
jNullj opened this issue Jan 8, 2022 · 3 comments
Closed

[ExplosmBridge] Call to a member function find() on null #2415

jNullj opened this issue Jan 8, 2022 · 3 comments
Labels
Bug-Report Confirmed bug report

Comments

@jNullj
Copy link
Contributor

jNullj commented Jan 8, 2022

Describe the bug
The ExplosmBridge "was unable to receive or process the remote website's content!"
Happens since 22/Dec, didnt change anything in my setup, it worked before that.

To Reproduce
Steps to reproduce the behavior:

  1. Use the ExplosmBridge feed

Expected behavior
Getting an rss feed the explosm comic.

Screenshots
image

Desktop (please complete the following information):

  • OS: Archlinux
  • Browser Chrome
  • Version 97
  • My rss client is TT-RSS

Additional context
Error message shows some i hope helpful info:

Error message: `Call to a member function find() on null`
Query string: `action=display&bridge=Explosm&format=Atom`
Version: `dev.2021-04-25`
@jNullj jNullj added the Bug-Report Confirmed bug report label Jan 8, 2022
@jNullj
Copy link
Contributor Author

jNullj commented Jan 8, 2022

Seems like they changed the site and div[id=comic-wrap] is not a thing anymore, i think i might be able to fix that by using the div tag with id=comic

@jNullj
Copy link
Contributor Author

jNullj commented Jan 8, 2022

I was able to fix the issue regarding the feed, but i noticed another issue, this bridge is using feedburner to retrive the comic list, and it appears that since the website was updated on 22/Dec feedburner stopped updating articles.
I think there is no rss option in the new site and another solution is required.

@jNullj
Copy link
Contributor Author

jNullj commented Jan 10, 2022

Solved with the merge #2417 , this was a bridge-braking issue.

@jNullj jNullj closed this as completed Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug-Report Confirmed bug report
Projects
None yet
Development

No branches or pull requests

1 participant