Skip to content

Revert "Use API routes for RSS feed (#153)"#157

Merged
shayne merged 1 commit intomainfrom
shayne/revert-api-routes-rss
Mar 28, 2023
Merged

Revert "Use API routes for RSS feed (#153)"#157
shayne merged 1 commit intomainfrom
shayne/revert-api-routes-rss

Conversation

@shayne
Copy link
Contributor

@shayne shayne commented Mar 28, 2023

This is causing the RSS feed to 500.

https://tailscale.dev/feed.xml

@vercel
Copy link

vercel bot commented Mar 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tailscale-dev ✅ Ready (Inspect) Visit Preview Mar 28, 2023 at 1:37AM (UTC)

@shayne shayne merged commit c0fa8dc into main Mar 28, 2023
@shayne shayne deleted the shayne/revert-api-routes-rss branch March 28, 2023 01:39
tylersmalley added a commit that referenced this pull request Mar 28, 2023
@rosszurowski
Copy link
Contributor

rosszurowski commented Mar 28, 2023

Thanks for the fix. Do you have the error message or logs from when it was failing? It worked fine in the preview deploy. Perhaps a difference between Vercel and Fly.io? (I don't think we've switched over the main site yet, right?)

@shayne
Copy link
Contributor Author

shayne commented Mar 28, 2023

It was only working locally for me. The (Vercel) preview and the (Fly) prod site were 500.

Here's that preview URL showing 500:
https://tailscale-q8h7w40qb-tailscale.vercel.app/feed.xml

@rosszurowski
Copy link
Contributor

Oh, huh? Maybe the kebab casing change broke something, since it was working before. I'll re-open the PR with a working setup. Sorry about that!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants