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

Authenticate client tracker with server #2

Open
martinteoharov opened this issue Mar 8, 2024 · 1 comment
Open

Authenticate client tracker with server #2

martinteoharov opened this issue Mar 8, 2024 · 1 comment

Comments

@martinteoharov
Copy link
Owner

We need a way to authenticate the client tracker script with the backend. I have no idea how to approach this, but for the time being we can just stick a ClientID in the URL that imports the tracker which we can append to any further requests.

In the future we will need a better way to authenticate users which will interface with the dashboard.

@martinteoharov martinteoharov converted this from a draft issue Mar 8, 2024
@martinteoharov martinteoharov changed the title Authenticate client tracker with server. Authenticate client tracker with server Mar 8, 2024
@lbfalvy
Copy link
Contributor

lbfalvy commented Mar 8, 2024

  • requires user consent in the EU
  • there's no point falsifying it so it doesn't have to be secure
  • processing a tracked event start to finish on the server should not be too complicated.

I think just an optional key provided by the page is fine, however, I don't think it's a good idea to assume how the tracker is loaded and it should definitely be possible to change this without a page reload when the user consents to identification.

The URL is a convenient place to pass this if the page uses SSR or loads the tracker from JS, but another option would be useful, such as a key on the global this object.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

2 participants