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

[DanbooruBridge] Decode href of HTML element to avoid double escaping. #1262

Merged
merged 1 commit into from Aug 26, 2019
Merged

[DanbooruBridge] Decode href of HTML element to avoid double escaping. #1262

merged 1 commit into from Aug 26, 2019

Conversation

ghost
Copy link

@ghost ghost commented Aug 17, 2019

Directly accessing ...->href resulted in a string that contained & instead of &. This was later escaped again to & in some
formats (e.g. Atom).

Fixes #1261 .

Directly accessing ...->href resulted in a string that contained '&'
instead of '&'. This was later escaped again to '&' in some
formats (e.g. Atom).
@ghost ghost changed the title Decode href of HTML element to avoid double escaping. [DanbooruBridge] Decode href of HTML element to avoid double escaping. Aug 17, 2019
@teromene teromene merged commit 6a99904 into RSS-Bridge:master Aug 26, 2019
@teromene
Copy link
Member

Thank you and welcome to the project !

@ghost ghost deleted the hotfix/1261_double_encoded_urls branch September 3, 2019 11:50
infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
RSS-Bridge#1262)

Directly accessing ...->href resulted in a string that contained '&'
instead of '&'. This was later escaped again to '&' in some
formats (e.g. Atom).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gelbooru bridge double encodes URLs
1 participant