Skip to content

add trailing slash where redirect occurs #577

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 7, 2025
Merged

add trailing slash where redirect occurs #577

merged 1 commit into from
Jul 7, 2025

Conversation

@BertrandRenault
Copy link
Contributor

Issue

sometime fetch cause a redirection because of a missing trailing '/' in url

Description

Add trailing slash for indentified redirections

@BertrandRenault BertrandRenault self-assigned this Jul 7, 2025
@BertrandRenault BertrandRenault added bug: minor Minor issues, minimal user impact scope: front Related to Pixano frontend labels Jul 7, 2025
@benlabbe benlabbe requested a review from antoningaladrim July 7, 2025 16:01
@BertrandRenault BertrandRenault requested review from benlabbe and removed request for antoningaladrim July 7, 2025 16:01
benlabbe
benlabbe approved these changes Jul 7, 2025
View reviewed changes
Copy link
Contributor

@benlabbe benlabbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hope this will fix the deployment behind an HTTPS reverse-proxy

@benlabbe benlabbe merged commit f943607 into main Jul 7, 2025
3 checks passed
@BertrandRenault BertrandRenault deleted the fix/avoid-redirects-trailing-slash branch July 25, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benlabbe benlabbe benlabbe approved these changes

Assignees

@BertrandRenault BertrandRenault

Labels

bug: minor Minor issues, minimal user impact scope: front Related to Pixano frontend

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@BertrandRenault @benlabbe