Skip to content
Snippets Groups Projects

don't double-url-encode rss urls

Merged dakkar requested to merge feature/875-fix-double-rss-url-encoding into develop

What does this MR do?

url.searchParams.set() already encodes the values passed!

(this is a partial revert of 0472d43e, the change in statusbar-rss.vue was correct)

@kakkokari-gtyih you probably want to cherry-pick this: the rss reader in the status bar works fine, the widgets don't, because they get something like https%3A%2F%2Flobste.rs%2Frss instead of https://lobste.rs/rss

Contribution Guidelines

By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this merge request

Merge request reports

Merge request pipeline #2257 passed

Merge request pipeline passed for b9e03339

Approved by

Merged by HazelnootHazelnoot 2 months ago (Jan 11, 2025 4:43pm UTC)

Merge details

  • Changes merged into develop with 771fcf00.
  • Deleted the source branch.

Pipeline #2258 passed

Pipeline passed for 771fcf00 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading