Skip to content

Improve SimplePie redirects (POST to GET, remove authentication cross-origin)#78

Open
Inverle wants to merge 4 commits into
FreshRSS:freshrssfrom
Inverle:improve-sp-redirects
Open

Improve SimplePie redirects (POST to GET, remove authentication cross-origin)#78
Inverle wants to merge 4 commits into
FreshRSS:freshrssfrom
Inverle:improve-sp-redirects

Conversation

@Inverle
Copy link
Copy Markdown
Member

@Inverle Inverle commented May 30, 2026

Comment thread src/File.php
@Inverle Inverle requested a review from Alkarex June 2, 2026 21:59
@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented Jun 5, 2026

Is it correctly understood that passing CURLOPT_FOLLOWLOCATION as a cURL option will allow skipping this new code?
I am interested in keeping an option to let cURL perform the work rather than using our or SimplePie's re-implementation

@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented Jun 5, 2026

Although we only use the cURL path in FreshRSS, the fsockopen path further below should probably be updated in the same manner

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.

2 participants