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

Redirection and URL rewrite #1162

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

Wonshtrum
Copy link
Member

Implement redirections and URL rewrite proposal of #1161.

Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
@Wonshtrum Wonshtrum force-pushed the devel/edemolis/feat/redirections branch from 6701c49 to 663b67f Compare December 17, 2024 20:30
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
@Wonshtrum Wonshtrum marked this pull request as ready for review December 19, 2024 14:47
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
@Wonshtrum Wonshtrum force-pushed the devel/edemolis/feat/redirections branch from 7b846d1 to 28ea2c6 Compare January 8, 2025 09:42
I feel like it's going nowhere...
Backend logic is messy and unclear, buffers are needlessly kept alive,
borrowing subsets of the HTTP session is harder and harder, reseting
default answers request is way too hard for what it's worth (it's
probably broken right now), gauges are desynched and we are cloning
again and again String ids...

Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
@Wonshtrum Wonshtrum force-pushed the devel/edemolis/feat/redirections branch from 28ea2c6 to 33513c4 Compare January 8, 2025 09:43
Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
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