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

Too eagerly redirecting to latest stable index page #95

Open
parlough opened this issue Dec 28, 2023 · 6 comments
Open

Too eagerly redirecting to latest stable index page #95

parlough opened this issue Dec 28, 2023 · 6 comments

Comments

@parlough
Copy link
Member

\cc @sortie

@sortie
Copy link
Contributor

sortie commented Dec 29, 2023

Yeah I can reproduce these bugs. Looking at the source, they don't appear to be new breakages caused by my recentish changes. As such, I won't treat it as a super important breakage and delay fixing these problems to the new year. It doesn't help that redirector.py is messy and hard to understand and correctly modify. :)

@sortie
Copy link
Contributor

sortie commented Dec 29, 2023

Like, I don't see any unit tests on this thing. It makes it risky to make changes especially since there's a production freeze going on.

@parlough
Copy link
Member Author

parlough commented Dec 29, 2023

As such, I won't treat it as a super important breakage and delay fixing these problems to the new year.

Thanks for checking in to this! No need to prioritize this then, even in the new year, I just wanted to make sure this was tracked.

Like, I don't see any unit tests on this thing. It makes it risky to make changes especially since there's a production freeze going on.

Yeah it's part of the reason I didn't just open a PR right away...It's in a pretty confusing state.

Rather than trying to make improvements and fixes, I keep thinking to ignore a bunch of the legacy handling and switch to some new setup, especially since app engine is planning to phase out Python 2.7 support. Anything new is quite hard for me to set up outside of Google though, but I'd be happy to help with any work in that area!

@sortie
Copy link
Contributor

sortie commented Feb 6, 2024

I tried fixing this, but honestly, the lack of testing made it really difficult and the code is really hard to understand.

Meanwhile the Python 2.7 apocalypse is on us. We're currently working towards replacing the project entirely with a new implementation using modern technologies. For now, let's just live with these older bugs.

@parlough
Copy link
Member Author

parlough commented Feb 6, 2024

The Python 2.7 apocalypse. The less-scary follow up to the Javapocalypse of 2013 😆

https://www.youtube.com/watch?v=E3418SeWZfQ

We're currently working towards replacing the project entirely with a new implementation using modern technologies.

I think that makes a lot of sense and is definitely a better use of time. I'll keep this open so I don't forget to verify the behavior in the future, but feel free to close.

If you have a chance, I'd be curious what setup you're exploring?

@sortie
Copy link
Contributor

sortie commented Feb 7, 2024

Basically we're thinking of having a cloud load balancer cache the contents right out of the storage bucket. We'll have to deal with the redirect logic somehow, maybe by deprecating it, or serving html http equiv refresh tokens, or by duplicating files with multiple paths.

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

No branches or pull requests

2 participants