Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Atom: invalid XML (Instagram, media:thumbnail) #1511

Closed
arokettu opened this issue Mar 30, 2020 · 0 comments · Fixed by #1566
Closed

Atom: invalid XML (Instagram, media:thumbnail) #1511

arokettu opened this issue Mar 30, 2020 · 0 comments · Fixed by #1566
Labels
Bug-Report Confirmed bug report

Comments

@arokettu
Copy link

Describe the bug
Instagram bridge produces invalid XML for Atom

To Reproduce
Steps to reproduce the behavior:

  1. Open instagram's Atom feed with some strict parser, Firefox should be enough
  2. Get xml error: XML Parsing Error: prefix not bound to a namespace

Expected behavior
Valid XML

Additional context
The problematic element is <media:thumbnail url="https://www.instagram.com/p/..."/>

To use it you should declare media namespace in the root element first:

<feed xmlns="http://www.w3.org/2005/Atom" xmlns:media="http://search.yahoo.com/mrss/">
...
</feed>
@arokettu arokettu added the Bug-Report Confirmed bug report label Mar 30, 2020
somini added a commit to somini/rss-bridge that referenced this issue Apr 12, 2020
@somini
Include Media RSS namespace for Atom feeds
44ef427 
Fix RSS-Bridge#1511

Fix RSS-Bridge#1499
This was referenced Apr 12, 2020
Closed
Open
@SuperSandro2000 SuperSandro2000 mentioned this issue May 18, 2020
Merged
teromene pushed a commit that referenced this issue May 19, 2020
@SuperSandro2000
[Core] Include Media RSS namespace for Atom feeds
8047041 
Include Media RSS namespace for Atom feeds
Fix #1511
Fix #1499
comlaterra pushed a commit to comlaterra/rss-bridge that referenced this issue Sep 25, 2020
@SuperSandro2000 @comlaterra
[Core] Include Media RSS namespace for Atom feeds
0ead0c9 
Include Media RSS namespace for Atom feeds
Fix RSS-Bridge#1511
Fix RSS-Bridge#1499
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug-Report Confirmed bug report
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Include Media RSS namespace for Atom feeds + fix tests
1 participant
@arokettu