Skip to content

Fix fetching OPML url with special characters#7843

Merged
Alkarex merged 3 commits intoFreshRSS:edgefrom
Inverle:opml-url-fix
Aug 25, 2025
Merged

Fix fetching OPML url with special characters#7843
Alkarex merged 3 commits intoFreshRSS:edgefrom
Inverle:opml-url-fix

Conversation

@Inverle
Copy link
Member

@Inverle Inverle commented Aug 20, 2025

Add dynamic OPML with url http://freshrss2.local/api/query.php?user=X&t=X&f=opml

Expect to see in server logs:

172.18.0.2 - - [20/Aug/2025:02:25:38 +0000] "GET /api/query.php?user=X&t=X&f=opml HTTP/1.1" 200 1591 "-" "FreshRSS/1.27.1-dev (Linux; https://freshrss.org)"

Instead this happens:

172.18.0.2 - - [20/Aug/2025:02:20:41 +0000] "GET /api/query.php?user=X&t=X&f=opml HTTP/1.1" 422 18 "-" "FreshRSS/1.27.1-dev (Linux; https://freshrss.org)"
image

@Alkarex Alkarex added this to the 1.28.0 milestone Aug 20, 2025
@Alkarex Alkarex added the Bug (confirmed) 🐞 issues that are reproducible label Aug 20, 2025
@Inverle Inverle changed the title Decode dynamic OPML url during fetch Fix fetching OPML url with special characters Aug 25, 2025
@Alkarex Alkarex merged commit 1481cf5 into FreshRSS:edge Aug 25, 2025
1 check passed
@Inverle Inverle deleted the opml-url-fix branch August 25, 2025 08:38
@Alkarex Alkarex modified the milestones: 1.28.0, 1.27.1 Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug (confirmed) 🐞 issues that are reproducible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants