as-ideas / oil

191 stars 56 forks source link

Add option to set the domain of the SOI cookie #244

Open cphilleo opened 5 years ago

cphilleo commented 5 years ago

The Problem

Multiple sites on sub-domains would like to share consent, e.g. sub1.example.com, sub2.example.com, sub3.example.com. This can be achieved by using the POI feature and specifying a common domain such as consent.example.com however there is overhead of at least 20K per page load to use POI.

Proposal

Since subdomains can set a shared cookie on the parent domain, it's more efficient to allow the SOI cookie to be set on the parent domain, and then be shared by all subdomains. This would be a form of group consent.

This pull request adds an option to set the domain to be used for the SOI cookie. Setting the cookie to an invalid non-parent domain results in no cookie being set.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 937


Files with Coverage Reduction New Missed Lines %
dist/oil.bundle.js 351 31.46%
<!-- Total: 351 -->
Totals Coverage Status
Change from base Build 936: 0.05%
Covered Lines: 1138
Relevant Lines: 2972

💛 - Coveralls
alexgit2k commented 4 years ago

Good suggestion, I would need the same. Can you please resolve the error.