didericis/bot-bottle
1
0
Fork 0
Code Issues 7 Pull Requests 15 Actions Packages Projects Releases Wiki Activity

fix(egress): strip Authorization before DLP scan #212

Merged
didericis merged 1 commits from fix/egress-strip-auth-before-scan into main 2026-06-07 22:36:01 -04:00
Conversation 0 Commits 1 Files Changed 3
+24 -17
didericis-claude commented 2026-06-06 16:15:42 -04:00
Collaborator
Copy Link
Copy Source

Summary

Moves Authorization stripping to happen before the DLP scan rather than after, and removes the auth_header parameter from scan_outbound.

The original order was: DLP scan (including auth header) → strip auth → route decision. The new order is: strip auth → DLP scan (body only) → route decision.

Merge rule(s)

Targets feat/egress-log-option; merge that branch first, then rebase this one to main.

## Summary Moves `Authorization` stripping to happen before the DLP scan rather than after, and removes the `auth_header` parameter from `scan_outbound`. The original order was: DLP scan (including auth header) → strip auth → route decision. The new order is: strip auth → DLP scan (body only) → route decision. ## Merge rule(s) Targets `feat/egress-log-option`; merge that branch first, then rebase this one to `main`.
didericis changed target branch from feat/egress-log-option to main 2026-06-07 20:32:49 -04:00
didericis added 1 commit 2026-06-07 22:30:18 -04:00
fix(egress): strip Authorization before DLP scan; remove auth_header param from scan_outbound
test / unit (pull_request) Successful in 32s
Details
test / integration (pull_request) Successful in 46s
Details
lint / lint (push) Successful in 1m27s
Details
test / unit (push) Successful in 35s
Details
test / integration (push) Successful in 42s
Details
Update Quality Badges / update-badges (push) Successful in 1m20s
Details
a04aed098d
didericis force-pushed fix/egress-strip-auth-before-scan from fcdcfe9a48 to a04aed098d 2026-06-07 22:30:18 -04:00 Compare
didericis merged commit a04aed098d into main 2026-06-07 22:36:01 -04:00
didericis deleted branch fix/egress-strip-auth-before-scan 2026-06-07 22:36:02 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
didericis-claude (didericis (claude)) didericis-codex (didericis (codex)) didericis-gemma (gemma) didericis
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: didericis/bot-bottle#212
Delete Branch "fix/egress-strip-auth-before-scan"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?