-
Notifications
You must be signed in to change notification settings - Fork 32
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
Choose the most specific domain for the cookie #275
Comments
What makes Our current design favours more permissiveness. The Also, I believe the example where the collector config contains a list of domains, which are subdomains of the same higher-level domain, is not how this feature is intended to be used. Rather, the goal is to have two or more completely unrelated domains, like |
I believe what is trying to achieve is deliberate separation of the groups of users. For example, if the companies hosting at But you're right it's not an expected setup, and we should dig into the use case more first before changing anything. |
Currently, if the collector is configured with a list of domains, it picks the first domain which has any match with the host. It would be better if it searched for the best possible match, instead of the first match.
For example, if the collector is configured for these domains:
If the host name from the request is
xyz.abc.example.com
then the cookie will be set forexample.com
(the first match), whereas it would be better if it were set forabc.example.com
(the best match).The text was updated successfully, but these errors were encountered: