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

Author name double escaped #1317

Closed
davidoskky opened this issue Jan 27, 2022 · 1 comment
Closed

Author name double escaped #1317

davidoskky opened this issue Jan 27, 2022 · 1 comment
Labels
Milestone

Comments

@davidoskky
Copy link
Contributor

davidoskky commented Jan 27, 2022

The HTML of the author name fetched is not correctly sanified.

For example, fetching with the youtube spout this channel: QualityAndTechnology
The author name displayed in Selfoss is "Chemometrics & Machine Learning in Copenhagen" instead of "Chemometrics & Machine Learning in Copenhagen"

@jtojnar jtojnar added the bug label Jan 27, 2022
@jtojnar jtojnar changed the title Author name not sanified Author name double escaped Oct 7, 2022
@jtojnar jtojnar modified the milestones: 2.19, 2.20 Oct 7, 2022
@jtojnar jtojnar closed this as completed in e153f2d Oct 9, 2022
@jtojnar
Copy link
Member

jtojnar commented Oct 9, 2022

Thanks for reporting. I have fixed it for newly fetched items. For the old one, you can run the SQL command mentioned in the commit message.

Unfortunately, this is not the only problematic item. Courtesy of Simplepie, almost every value returned by the library (except for URIs) has HTML special characters escaped, if it is a plain text and will require a larger revamp (I started on it in #1368 but we do not want it to delay 2.19).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants