mirror of
https://github.com/gotempsh/temps.git
synced 2026-04-25 06:15:55 +03:00
[PR #4] [MERGED] fix(proxy): remove internal ID headers from proxy responses #6
Labels
No labels
bug
enhancement
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/temps#6
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
📋 Pull Request Information
Original PR: https://github.com/gotempsh/temps/pull/4
Author: @dviejokfs
Created: 2/16/2026
Status: ✅ Merged
Merged: 2/16/2026
Merged by: @dviejokfs
Base:
main← Head:feat/remove-internal-id-response-headers📝 Commits (1)
6768f6dfix(proxy): remove internal ID headers from proxy responses📊 Changes
2 files changed (+19 additions, -11 deletions)
View changed files
📝
README.md(+19 -1)📝
crates/temps-proxy/src/proxy.rs(+0 -10)📄 Description
Summary
X-Project-ID,X-Environment-ID, andX-Deployment-IDheaders from proxy responses — these leaked sequential internal integer IDs to end users, enabling enumeration and exposing potential IDOR attack surfaceX-Request-IDfor request tracing — operators can correlate with server-side logs for full routing contextSecurity
The removed headers exposed internal database IDs (sequential integers) on every proxied response. This allowed external observers to:
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.