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.

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

Support URI field for Author in Atom Feed #200

Open
imirzadeh opened this issue Feb 25, 2023 · 3 comments
Open

Support URI field for Author in Atom Feed #200

imirzadeh opened this issue Feb 25, 2023 · 3 comments

Comments

@imirzadeh
Copy link

Expected behavior

According to the specifications, author in Atom can have uri. This is useful for blog platforms where in the returned xml does not return html link to the webpage and instead returns author's profile/webpage. Here's an example

Actual behavior

The Person in gofeed has only name and email. But for Atom feeds, it needs uri as well.

@mmcdole
Copy link
Owner

mmcdole commented Feb 28, 2023

@cristoper before I check on this, did your latest change address this? I briefly saw some handling of URI components in Atom, but wasn't sure if it was unrelated.

@cristoper
Copy link
Contributor

No, this is unrelated.

It looks like atom.Person does have a URI field, but gofeed.Person does not.

@mmcdole
Copy link
Owner

mmcdole commented Mar 25, 2023

Alright, I don't see an issue with us adding URL to gofeed.Person and map atom.Person.URI to it.

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

No branches or pull requests

3 participants