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

Disable FLoC by default #1263

Closed
Rich-Harris opened this issue Apr 28, 2021 · 4 comments · Fixed by #1267
Closed

Disable FLoC by default #1263

Rich-Harris opened this issue Apr 28, 2021 · 4 comments · Fixed by #1267

Comments

@Rich-Harris
Copy link
Member

Is your feature request related to a problem? Please describe.
FLoC is bad, and I think we should join the many organisations that treat it as a security/privacy threat and disable it by default.

Describe the solution you'd like
Add a new kit.floc config option that defaults to false, and which must be set to true to opt in to FLoC:

module.exports = {
  kit: {
    floc: true
  }
};

If false, the following header gets added to every response:

Permissions-Policy: interest-cohort=()

Describe alternatives you've considered

  • Doing nothing and leaving it to app developers. This places a significant burden on them; while the userland implementation is straightforward, we can't reasonably expect most people to be aware of this issue
  • Making it opt out rather than opt in. This reduces the implementation burden, but not the awareness burden

SvelteKit is, among other things, a statement about what kind of web we want to exist. I think it's entirely appropriate to treat this as being within the framework's purview, as much as providing tools to mitigate against things like CSRF attacks.

How important is this feature to you?
In practical terms, not very. In terms of principle, I think it's important for the reasons given above.

@Conduitry
Copy link
Member

👍 Full support for this.

Rich-Harris added a commit that referenced this issue Apr 28, 2021
@Rich-Harris Rich-Harris mentioned this issue Apr 28, 2021
5 tasks
Rich-Harris pushed a commit that referenced this issue Apr 29, 2021
* implement floc option - closes #1263

* changeset

* docs
@ebeloded
Copy link

ebeloded commented May 1, 2021

Is it expected to have this warning now?
image

@frederikhors
Copy link
Contributor

Is it expected to have this warning now?
image

I am always of the opinion that a default browser warning is wrong.

@100lvlmaster
Copy link

100lvlmaster commented Jun 14, 2021

Is it expected to have this warning now?
image

I am always of the opinion that a default browser warning is wrong.

This man came back to this issue, to make that joke. Commendable

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 a pull request may close this issue.

5 participants