-
Notifications
You must be signed in to change notification settings - Fork 316
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
Timetable share URL is no longer shortened #2057
Comments
Oh, it's a bug... I actually liked it. |
@angelsl I'm interested in why you like it. Could you explain a little why you prefer the long url? |
@li-kai Sometimes I want to merge multiple timetables into one, etc; the long URL lets me do that easily. Also saving the long URL gives me a bit more security since the information is there, versus the shortened URL where if the server dies my timetable would be lost. I mean I could just resolve the shortened URL, but it's always one more step. |
Describe the bug
The URL should've been shortened to something like
https://modsn.us/<somehash>
.Cause
We have a short_url.php script which mediates requests to our modsn.us URL shortener service. This was previously supported by Nginx. However, as #1937 changes the reverse proxy to Traefik, we need to add back PHP support for this to work again.
Possible actions
Add fastcgi support to our Traefik reverse proxy.No longer needed as we've migrated away from Docker (Migrate /website and /export from Docker to Vercel #3098)enableShortUrl
feature flag once done.The text was updated successfully, but these errors were encountered: