Skip to content

fix redirectOnDirectory#28

Open
ptrmsk wants to merge 1 commit intojgkaplan:masterfrom
ptrmsk:redirectOnDirectory-fix
Open

fix redirectOnDirectory#28
ptrmsk wants to merge 1 commit intojgkaplan:masterfrom
ptrmsk:redirectOnDirectory-fix

Conversation

@ptrmsk
Copy link
Contributor

@ptrmsk ptrmsk commented Feb 19, 2023

No redirects were happening when browsing a static folder without a trailing backslash, despite the flag. (gemini://site/staticFolder for example, wouldn't redirect to /staticFolder/). This broke relative links

…i://site/staticFolder for example). Broke relative links in child pages
@ptrmsk
Copy link
Contributor Author

ptrmsk commented Feb 19, 2023

This redirect strategy/fix doesn't work on all clients (Android's Buran is a big outlier here), but it works on many others (Lagrange, Deedum, Xenia). I'm guessing this is implementation-specific and not the server's fault.

I also noticed going to "gemini://site" causes issues, but that might be outside the scope of this

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.

1 participant