SimplePie fix parsing of HTTP Links#4283
Conversation
* https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Link * https://datatracker.ietf.org/doc/html/rfc8288 Before, SimplePie was not able to parse something like ``` Link: <https://pubsubhubbub.appspot.com>; rel="hub", <https://pubsubhubbub.superfeedr.com>; rel=hub, <https://websubhub.com/hub>; rel="hub" ```
|
I'll give it a go later, but I suspect edge + those two or three commits thrown in, served using the Docker container ( |
|
To test WebSub, FreshRSS needs to be deployed with a routable address |
|
IP with the right port exposed should suffice, right? |
|
Yes, should be |
|
Will it fix #4257 too? |
|
Not just fix, that's the issue. 😁 But yes, we need to double check it's parsed properly now. |
Ah indeed :-) I did not see that issue (I am a bit in late with my notifications) so thanks for linking |
|
I checked. It will fix the issue 4257 |
|
I have tested with several feeds and I could not find any regression. Please confirm when you can, @Frenzie :-) |
|
I don't think I see any issues. |
Before, SimplePie was not able to parse an HTTP header such as: ``` Link: <https://pubsubhubbub.appspot.com>; rel="hub", <https://pubsubhubbub.superfeedr.com>; rel=hub, <https://websubhub.com/hub>; rel="hub" ``` The new regex complies with https://datatracker.ietf.org/doc/html/rfc8288 See also https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Link Downstream: FreshRSS/FreshRSS#4283
|
Upstream: simplepie/simplepie#729 |
Before, SimplePie was not able to parse an HTTP header such as: ``` Link: <https://pubsubhubbub.appspot.com>; rel="hub", <https://pubsubhubbub.superfeedr.com>; rel=hub, <https://websubhub.com/hub>; rel="hub" ``` The new regex complies with https://datatracker.ietf.org/doc/html/rfc8288 See also https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Link Downstream: FreshRSS/FreshRSS#4283
@fix #4279 (part of)
Before, SimplePie was not able to parse an HTTP header such as: