Skip to content

Fixing flash message cookie handling #21

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
angrynode merged 1 commit into from
Dec 1, 2025
Merged

Fixing flash message cookie handling #21

angrynode merged 1 commit into from
Dec 1, 2025

Conversation

@Gabatxo1312
Copy link
Collaborator

@Gabatxo1312 Gabatxo1312 commented Dec 1, 2025

No description provided.

@Gabatxo1312 Gabatxo1312 self-assigned this Dec 1, 2025
@Gabatxo1312 Gabatxo1312 requested a review from angrynode December 1, 2025 13:53
@angrynode
Copy link
Owner

angrynode commented Dec 1, 2025

For further code historians, this is because if you don't set the path on the cookie, it's only valid on the current route, not on the entire domain. Which means if your POST submission endpoint is in a different route, the cookie won't appear!

@angrynode
Copy link
Owner

angrynode commented Dec 1, 2025

That's perfect thank you!

@angrynode angrynode merged commit 191945b into angrynode:axum Dec 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@angrynode angrynode Awaiting requested review from angrynode

Assignees

@Gabatxo1312 Gabatxo1312

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Gabatxo1312 @angrynode