Skip to content

Scrub UUID where requested - #15694

Open
SabreCat wants to merge 3 commits into
developfrom
kalista/deid-logs
Open

Scrub UUID where requested#15694
SabreCat wants to merge 3 commits into
developfrom
kalista/deid-logs

Conversation

@SabreCat

@SabreCat SabreCat commented Aug 7, 2026

Copy link
Copy Markdown
Member

Habitica files various error and informational messages to Loggly, a third-party application monitoring platform.

Before, these messages always include the user's ID in the x-api-user header field, which could be used to uniquely identify the originator of recorded app activity.

Now, if the user has not given consent for their data to be recorded to third-party platforms, we scrub this header after authenticating that it corresponds to a valid Habitica account. Should an error or other message be logged after this point in server request handling, the x-api-user field will simply show "Private".

@SabreCat
SabreCat force-pushed the kalista/deid-logs branch from a516526 to 0721756 Compare August 7, 2026 22:33
@SabreCat
SabreCat marked this pull request as ready for review August 7, 2026 22:49
@SabreCat
SabreCat requested a review from Hafizzle August 7, 2026 22:49

@Hafizzle Hafizzle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

checked that the header change doesn't affect rate limiting, lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants