Skip to content

Restrict debugging#212

Merged
moseshll merged 4 commits intomainfrom
restrict_debugging
Feb 23, 2026
Merged

Restrict debugging#212
moseshll merged 4 commits intomainfrom
restrict_debugging

Conversation

@moseshll
Copy link
Collaborator

@moseshll moseshll commented Feb 20, 2026 •
edited
Loading

Based on an internal discussion. Highlights of this PR (the first bullet point is the important one):

  • Restrict debugAuth debugging output to internal table
    • Redirect immediately to wayf when access is unauthenticated
  • Restrict remaining debug print-to-HTML functionality to development instance (dev VMs and Docker)
  • Remove support for minimally-used debugVar URL param

Available for testing on dev-2

@moseshll moseshll marked this pull request as ready for review February 20, 2026 20:22
@moseshll moseshll requested a review from aelkiss February 20, 2026 20:22
aelkiss
aelkiss approved these changes Feb 23, 2026
View reviewed changes
Copy link
Member

@aelkiss aelkiss left a comment

Choose a reason for hiding this comment

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

This all looks reasonable to me. I didn't dig into it to see what was in the debug output before, but it makes sense to limit the amount of debugging info it's possible to get from a production request.

@moseshll moseshll merged commit 9fbee5b into main Feb 23, 2026
1 check passed
@moseshll moseshll deleted the restrict_debugging branch February 23, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aelkiss aelkiss aelkiss approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@moseshll @aelkiss