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

Using atom elements in RSS feed #151

Open

Necoro opened this issue Aug 5, 2020 · 1 comment · May be fixed by #153

Contributor

Necoro commented Aug 5, 2020

Stackoverflow uses Atom's author tags in their RSS feed (cf.
https://stackoverflow.com/jobs/feed). They indeed use the correct namespace for it. But currently, as this namespace is unknown, it is just added as a random extension.

My wishful expected behavior would be to fill the author information of the RSS-feed structure.

I could implement it, but I want to ask beforehand:

Should this be done on a case-by-cases basis, read: for the moment only for the author tag.
In a general way: Allow atom tags in all possible places. This will probably turn out to be quite a lot of work (and messy?)
Ignore it, chimera feeds are evil :)

Necoro mentioned this issue

StackOverflow: Atom author tag in RSS feed Necoro/feed2imap-go#24

Closed

Contributor Author

Necoro commented Aug 5, 2020

Ok, special atom handling in RSS is already done for link, so I'll do the same for author.

While at it, I'll incorporate atom and atom03 into canonicalNamespaces to avoid having to guess by namespace prefix.

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Add atom namespaces to known namespaces

2bfd1fb

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Atom parser: Avoid superfluous intermediate variables

2942cc1

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Test case with atom author

dc4b4fe

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Allow additional parsers for feed formats. Currently onl…

fac99d8

…y atom is allowed as part of RSS

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

54d4e6c

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

643bc77

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

b4788a4

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

6835f06

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

d909059

Necoro linked a pull request

that will close this issue

Allow parsing of atom tags in RSS #153

Open

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Add atom namespaces to known namespaces

eb079eb

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Atom parser: Avoid superfluous intermediate variables

bc4be48

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Test case with atom author

bd17cb1

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Allow additional parsers for feed formats. Currently onl…

4a2c11d

…y atom is allowed as part of RSS

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

be23901

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

a228bb7

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

5efeb73

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

17abd35

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

27cf04a

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Add atom namespaces to known namespaces

13b38ac

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Atom parser: Avoid superfluous intermediate variables

5a5dca8

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Test case with atom author

dd3bd15

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Allow additional parsers for feed formats. Currently onl…

9976a4b

…y atom is allowed as part of RSS

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

56fd457

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

7057c57

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

fcd4f80

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

68f7504

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

dc1f362

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

9eb41f5

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

8cab161

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

69a2fc6

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

03c4299

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

719f3c5

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Add atom namespaces to known namespaces

c878b4e

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Atom parser: Avoid superfluous intermediate variables

9581d68

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Test case with atom author

068281a

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Allow additional parsers for feed formats. Currently onl…

3c6e1eb

…y atom is allowed as part of RSS

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

7d15669

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

78062da

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

8fa89b5

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

4a69e32

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Add atom namespaces to known namespaces

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Atom parser: Avoid superfluous intermediate variables

dfc82d0

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Allow additional parsers for feed formats. Currently onl…

f63bd72

…y atom is allowed as part of RSS

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

5cbfff0

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

118b8ec

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

dde77cd

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

442c185

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

fca674b

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Add atom namespaces to known namespaces

c7484c4

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Atom parser: Avoid superfluous intermediate variables

6a2266c

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Allow additional parsers for feed formats. Currently onl…

716142d

…y atom is allowed as part of RSS

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: To make the test work, the parsed extensions have to tak…

a9cfebf

…e a json roundtrip

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Disable extension support for channels

d7dda83

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Translation support for parsed extensions

4ed852f

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151 Testcase for atom author in RSS

70c9e42

Necoro added a commit to Necoro/gofeed that referenced this issue


          mmcdole#151: Support for atom update date

932fea1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment