Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

RSS/Atom Feeds should use absolute URLs to be standards-compliant #14133

Closed
agentofuser opened this issue May 18, 2019 · 7 comments
Closed

RSS/Atom Feeds should use absolute URLs to be standards-compliant #14133

agentofuser opened this issue May 18, 2019 · 7 comments
Labels
topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label

Comments

@agentofuser
Copy link
Contributor

Description

I found a ("hacky", as he himself describes it) workaround on Dan Abramov's blog:

gaearon/overreacted.io#65

The workaround is based on string replacement though, which would replace false-positives such as matches within code blocks. I'm also not sure whether href and src cover every use of urls in the html spec.

Steps to reproduce

Expected result

  • href="https://gatsby-starter-blog-demo.netlify.com/static/8058f3f26913fea3b6a89a73344fe94a/2b40c/salty_egg.jpg"
  • src="https://gatsby-starter-blog-demo.netlify.com/static/8058f3f26913fea3b6a89a73344fe94a/c739e/salty_egg.jpg"

Actual result

  • href="/static/8058f3f26913fea3b6a89a73344fe94a/2b40c/salty_egg.jpg"
  • src="/static/8058f3f26913fea3b6a89a73344fe94a/c739e/salty_egg.jpg"
@freiksenet freiksenet added status: confirmed Issue with steps to reproduce the bug that’s been verified by at least one reviewer. type: bug An issue or pull request relating to a bug in Gatsby labels May 21, 2019
@freiksenet
Copy link
Contributor

Hi!

Thanks a lot for filing this issue! I wonder if you would like to make a PR to gatsby-plugin-feed to fix that?

Thank you! 💜

@freiksenet freiksenet added the help wanted Issue with a clear description that the community can help with. label May 21, 2019
@agentofuser
Copy link
Contributor Author

@freiksenet you're welcome :) I'd love to help, yes. I will probably have time to write that PR in a week or two if waiting that long is ok.

@gatsbot gatsbot bot added the stale? Issue that may be closed soon due to the original author not responding any more. label Jun 13, 2019
@gatsbot
Copy link

gatsbot bot commented Jun 13, 2019

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.

If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!

As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

@freiksenet freiksenet added not stale and removed stale? Issue that may be closed soon due to the original author not responding any more. labels Jun 17, 2019
@humphreybc
Copy link
Contributor

Ran into this one today. @agentofuser how did you get on with the PR?

@atuttle
Copy link

atuttle commented Oct 9, 2019

I'd be happy to take a crack at it, but I don't understand the code well enough. Can anyone lend a bit of insight to help get me started?

@LekoArts LekoArts added topic: rss-feed type: feature or enhancement Issue that is not a bug and requests the addition of a new feature or enhancement. and removed type: bug An issue or pull request relating to a bug in Gatsby labels Jul 3, 2020
@eubanksmd
Copy link

Found a great blog article that describes how to replace the relative URL with absolute URL in the RSS feed while this issue gets sorted out:
https://markshust.com/2020/06/25/fixing-images-in-gatsby-rss-feeds/

@LekoArts LekoArts added topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label and removed topic: rss-feed labels May 28, 2021
@bytrangle
Copy link
Contributor

bytrangle commented Jul 19, 2021

Hello @LekoArts ,
Do you still need this issue to be solved? If so, I'd like to take a crack at it.

@LekoArts LekoArts removed help wanted Issue with a clear description that the community can help with. status: confirmed Issue with steps to reproduce the bug that’s been verified by at least one reviewer. not stale type: feature or enhancement Issue that is not a bug and requests the addition of a new feature or enhancement. labels Aug 9, 2021
@gatsbyjs gatsbyjs locked and limited conversation to collaborators Aug 9, 2021
@LekoArts LekoArts closed this as completed Aug 9, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants