Skip to content

Comments

Don't set session cookies on indexed asset requests#5503

Merged
Floppy merged 1 commit intomainfrom
no-session-on-assets
Feb 19, 2026
Merged

Don't set session cookies on indexed asset requests#5503
Floppy merged 1 commit intomainfrom
no-session-on-assets

Conversation

@Floppy
Copy link
Collaborator

@Floppy Floppy commented Feb 19, 2026

In order to avoid session leakage via caches.

In order to avoid session leakage via caches.
@codacy-production
Copy link

codacy-production bot commented Feb 19, 2026

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for b58e2921 100.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (b58e292) Report Missing Report Missing Report Missing
Head commit (339321c) 6124 5177 84.54%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#5503) 1 1 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

@Floppy Floppy merged commit 8d30b93 into main Feb 19, 2026
16 checks passed
@Floppy Floppy deleted the no-session-on-assets branch February 19, 2026 15:27
@github-project-automation github-project-automation bot moved this from Maybe to Done in Manyfold Roadmap Feb 19, 2026
@github-project-automation github-project-automation bot moved this from Maybe to Done in Manyfold Development Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working security

Projects

Development

Successfully merging this pull request may close these issues.

1 participant