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

Blog RSS workaround via redirect #5520

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Nov 4, 2024

@chalin chalin added the CI/infra CI & infrastructure label Nov 4, 2024
@chalin chalin requested a review from a team as a code owner November 4, 2024 15:19
@opentelemetrybot opentelemetrybot requested a review from a team November 4, 2024 15:19
@github-actions github-actions bot added the blog label Nov 4, 2024
@chalin chalin mentioned this pull request Nov 4, 2024
Copy link
Member

@svrnm svrnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

@chalin chalin merged commit d3c66e3 into open-telemetry:main Nov 4, 2024
17 checks passed
@chalin chalin deleted the chalin-im-rw-blog-rss-workaround-2024-11-04 branch November 4, 2024 15:52
@chalin
Copy link
Contributor Author

chalin commented Nov 4, 2024

Hopefully this will work.
@dmathieu - can you confirm?

@jord1e
Copy link
Contributor

jord1e commented Nov 4, 2024

Wont 301 permanently redirect to /2024 now? (At least until the feed readers’ http cache resets, no idea what the limit is on that)

Although most readers will have already picked up the change 🥲

@dmathieu
Copy link
Member

dmathieu commented Nov 5, 2024

I can confirm this works for me. However, I agree with @jord1e, the redirect should be a 302.
With a 301, a feed reader could decide to permanently change the feed URL to be that of 2024.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog CI/infra CI & infrastructure
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants