-
Notifications
You must be signed in to change notification settings - Fork 616
Firebase Analytics: Setting userID to null does not clear the userID #3602
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
Comments
I found a few problems with this issue:
|
Hi @zmeid, thanks for reporting. I was able to reproduce the same issue. That being said, I'll notify an engineer about this. |
Hi @zmeid, upon investigating, it looks like the issue isn't the setUserId not working. The user id is indeed removed, but the debugView isn't showing this, so this is a UI issue. There's a similar issue in firebase/firebase-ios-sdk#4856 |
Hi @argzdev. Thanks for quick investigation! Should we continue to watch that issue on firebase/firebase-ios-sdk#4856? |
Hi @zmeid, this is currently being tracked in b/149401786. Yes, please do continue to monitor that issue as well. I'll add replies back here once we get updates. Thanks! |
This came up again (third time now...) in invertase/react-native-firebase#6583 |
Hi @mikehardy, I checked the internal bug and unfortunately there hasn't been any progress. Let me bump up the internal bug to get some updates. |
I just implemented a new (actually working!) stale bot on react-native-firebase and it showed me this had gone stale over there, so I'm carrying the same "is this stale? is there something to do?" message over to here. Any news ? Thanks! |
Hi all. Sorry for the silence here. Our engineers are still looking into how to fix the issue. Though rest assured, this is only a UI issue in DebugView, and the user ID is indeed removed once it is set to null. Also something we noticed during experimentation is that while the DebugView UI issue still shows up, this does clear up after approx. 1 hour. |
Anything new on this topic? We are running into the same issue |
Similar issue for GA4 analytics |
Still the issue |
Any updates on this? This makes every app using the firebase/analytics in breach of the EU GDPR & ePR laws. As clearing the userID still records in GA4/Firebase |
I facing this issue too, any update here? |
@mvn-vinhhuynh-dn Seems like this would need some work from the community to get it up and running. We've done a work around for now. Wiping the userId, then creating a new session when wiped. This removes the userId completely from. Not ideal, but works. |
this mean we set
|
Yes, in theory, however we ended up using: |
aw, thanks for answering @instesti |
Still not fixed |
In order to restart the session you can do: analytics().setAnalyticsCollectionEnabled(true);
analytics().setUserId(userId) // this can be skipped if you want to restart the session without userId, given it's already wiped |
@mikehardy Do you have any ideas if this will ever be addressed by G? (I can reach out to my contact to try to get some info, unless you aware of something). As mentioned in the earlier comments, we've done a workaround which is not pretty, but makes sense that the underlying issue is fixed from G's side. |
@instesti I have no insight, sorry |
[REQUIRED] Step 2: Describe your environment
[REQUIRED] Step 3: Describe the problem
It is stated in the docs that setting the userID to null should remove the userID. But, it does not remove the user ID when it is set to null.
Steps to reproduce:
That's a big privacy issue. As a workaround, when user chooses to opt out from identified analytics, we set the userID to String "null". So, userID gets updated to String "null".
Some other related threads:
The text was updated successfully, but these errors were encountered: