Skip to content
Snippets Groups Projects

security: document and explain session CSRF safety better

Merged Warren Gifford requested to merge sg/public-api-refactor-4 into main

Created by: slimsag

Stacked on top of #27298

The docstring already does a good job of explaining what this function does, but it was left up to the reader to then apply that logic to the actual code. Instead, we should also make the logic flow documented so that it is easy to follow, and easy to reason about changes to this code in the future.

Signed-off-by: Stephen Gutekanst stephen@sourcegraph.com

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading