Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The X-USER-STAGING-ID header electron-builder sends is a privacy violation #7728

Closed
fortuna opened this issue Aug 12, 2023 · 3 comments
Closed
Labels

Comments

@fortuna
Copy link

fortuna commented Aug 12, 2023

Whenever electron-builder fetches the update file, it sends a X-USER-STAGING-ID header that uniquely identifies the user. That is a privacy violation that affects any Electron app using electron-builder. See code at

client.setRequestHeaders(this.computeFinalHeaders({ "x-user-staging-id": stagingUserId }))

It looks like it was added back in 2017 to support some advanced version of staged rollouts.

Please disable the X-USER-STAGING-ID by default and don't enable it without explicit developer action. You don't need that for staged rollouts.

@mmaietta
Copy link
Collaborator

mmaietta commented Sep 1, 2023

Can you elaborate on how that is a privacy violation?

Unfortunately, disabling it by default is a breaking change, so that doesn't feel like we can easily take that approach. The final headers still will allow overrides via this.requestHeaders being set, so if you were to provide x-user-staging-id: "", that seems like you'd resolve your privacy issue?

Copy link
Contributor

github-actions bot commented Nov 1, 2023

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the Stale label Nov 1, 2023
Copy link
Contributor

This issue was closed because it has been stalled for 30 days with no activity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants