-
Notifications
You must be signed in to change notification settings - Fork 2
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
Some entries doesn't load metadata
properly
#14
Comments
Thank you for the report. I'll look into that. |
I was trying to download the XML file from the RSS feed. But for some reason my wget command does not work anymore. Do you happen to have either a working wget/curl command or the files? The former is preferred since I could reproduce that. |
Here you go, I just renamed them as .log otherwise Github doesn't accept the .xml |
Thank you. Do you have a way of downloading them? I'll be interested since I am hitting 403 errors when using wget. |
Just opened them in the browser and saved the page 😄 |
You're lucky. When I am doing that it enters in an non-ending loop of downloading atom files. |
Sometimes, some feed entries doen't display properly and are skipped by the plugin.
Example :
https://www.reddit.com/user/neo3dofficial/submitted/.rss?sort=new
At some point this error happens (
NO METADATA
is a log I added in theisValid
method) :If I make the metadata regex a bit more tolerent with
#(?P<metadata>\s+submitted.*</span>)#
, the error is gone.This allows the entry to actually be processed by the transformers and the image inlined. Before, as the error happened, the entry was skipped from going through the processors.
However this doesn't seem to handle all cases.
Example (NSFW) :
https://www.reddit.com/user/throwmeaway896/submitted/.rss?sort=new
With this feed, even if I have the regex modified, some entries failed to match (though image was already added by the BeforeInsertProcessor) :
I have to say I don't really understand that one, using an online checker the regex seems to match https://www.phpliveregex.com/p/JSm
The text was updated successfully, but these errors were encountered: