Fix Atom feed #764
No reviewers
Labels
No labels
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: Plume/Plume#764
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "fix-atom"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This patches include breaking change. I percent-encoded URI segments in
ap_url
s. This affects only future users, blogs and posts. Existingap_url
s will not be changed.This fixes Atom feeds so that it conform to Atom spec. This should fix Atom-related issues #673 and #735.
Codecov Report
Could it be possible to encode URI only for the Atom feed, not in the whole app? The JSON-LD spec (on which ActivityPub is built) says that every URI actually is an IRI, meaning it should be able to contain any Unicode character (with very few exceptions), so I don't know how percent-encoded strings are treated…
I've done! Thank you for your advice.
And... actually, Atom also uses IRI. Its spec says:
This is the reason why I reverted all the code around percent-encoding. I don't know why W3C feed validator claims IRIs are errors. But, how about deploying IRI version of Atom and convert them to URI if someone reports issue about specified feed reader implementation?
Sorry, but I'm not sure if I understand. Could you please try to rephrase ? (my English is so bad, help) ☹️
Thank you for reply. I'm also not good at English. My English should be hard to understand. Will rephrase it. What I mean was...
id
,link
and so on.Okay, it is clearer, thank you. Indeed, I think we can merge.
Thank you!