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

Posts with incorrect encoding #89

Closed
arzoriac opened this Issue Jul 25, 2018 · 7 comments

Comments

Projects
None yet
3 participants
@arzoriac
Copy link

arzoriac commented Jul 25, 2018

The encoding for Scripting.com is being set incorrectly. Take a look at the screenshots below to see how it is being delivered and displayed. The first one is from Python's dominant and wildly used feedparser module, the other is how it looks on NewsBlur.

feedparser

newsblur

@scripting

This comment has been minimized.

Copy link
Owner

scripting commented Jul 25, 2018

I linked to this from scripting.com. Hopefully other people will be able to help by telling us how it looks in their readers.

@brentsimmons

This comment has been minimized.

Copy link

brentsimmons commented Jul 25, 2018

screen shot 2018-07-25 at 3 05 23 pm

Looks fine in my RSS reader Evergreen. Also looked at the source in BBEdit, and used xmllint on the file. All looks good to me.

@brentsimmons

This comment has been minimized.

Copy link

brentsimmons commented Jul 25, 2018

If the top of the feed had <?xml version="1.0" encoding="UTF-8"?> — if it set the encoding there — then it would probably work fine.

@arzoriac

This comment has been minimized.

Copy link
Author

arzoriac commented Jul 25, 2018

The culprit is probably Python's feedparser.

@scripting

This comment has been minimized.

Copy link
Owner

scripting commented Jul 25, 2018

@scripting

This comment has been minimized.

Copy link
Owner

scripting commented Jul 26, 2018

The feed has been updated. @arz could you try running your test again and report the result.

http://xmlviewer.scripting.com/?url=http://scripting.com/rss.xml

Dave

@arzoriac

This comment has been minimized.

Copy link
Author

arzoriac commented Jul 26, 2018

Hooray! It works! Thank you Dave & Brent!

screen shot 2018-07-25 at 22 44 54

@scripting scripting closed this Jul 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.