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

Android 12 tells me that "Mozilla VPN pasted from your clipboard", for no obvious reason #2183

Closed
dholbert opened this issue Nov 2, 2021 · 7 comments · Fixed by #2280
Closed
Assignees

Comments

@dholbert
Copy link
Collaborator

dholbert commented Nov 2, 2021

Preconditions: Have a phone running Android 12, with Mozilla VPN installed but not currently running. (I've seen this when it was already running, too, but it's more predictable if it's not.)

STR:

  1. Copy a URL to your clipboard.
  2. Open the Mozilla VPN app.

ACTUAL RESULTS:
Immediately, I get a notification "Mozilla VPN pasted from your clipboard".

EXPECTED RESULTS:
No such notification. There's no reason Mozilla VPN should be inspecting my clipboard (particularly without any interaction on my part besides opening the app).

Screenshot:
image

See related Fenix bug: mozilla-mobile/fenix#22271
...though this notification is more understandable on Fenix, because over there, this is usually associated with some "fill link from clipboard" UI there, which means users can understand why they're seeing the notification. Whereas in Mozilla VPN, there's no clear reason for the app to be reading the user's clipboard (particularly at startup before any user interaction).

┆Issue is synchronized with this Jira Bug

@dholbert
Copy link
Collaborator Author

dholbert commented Nov 2, 2021

Phone model: Pixel 4a
Mozilla VPN App version: 2.5.1

@strseb
Copy link
Collaborator

strseb commented Nov 3, 2021

Hey! Are you running by chance a pre-release build of android 12? I was able to experience it on my pixel 4 on preview too, but it stopped happening after the latest update.
Also, we have no code accessing the clipboard, which just makes this extra weird 🙈

@dholbert
Copy link
Collaborator Author

dholbert commented Nov 3, 2021

I'm not running a pre-release build, no. I just installed the regular Android 12 release when it became available (via tapping Settings App, System, System Update)

@dholbert
Copy link
Collaborator Author

dholbert commented Nov 3, 2021

I just received a ~24mb "security update" to Android 12 (so now System Update shows me as being at version Android security update: November 5, 2021. I retested and confirmed that I can still reproduce.

@strseb: two important factors which impact reproducibility:
(1) When testing, do you have a URL that you've just copied to your clipboard? (not one that's been sitting in your clipboard for a while). It needs to be recently copied, or at least new-to-the-clipboard since the last time Mozilla VPN was running, I think.
(2) Are you starting Mozilla VPN "fresh" (if it was previously running, you need to swipe it away in the app-switcher to kill it)?

I can reproduce this system-notification 100% of the time, with these two factors. (Specifically, I'm long-pressing a link in an email and choosing "Copy URL"; and then I start Mozilla VPN, without it having been previously running. I immediately see "Mozilla VPN pasted from your clipboard" when the app launches, essentially as part of the initial paint of the app.)

@dholbert
Copy link
Collaborator Author

@strseb just checking in -- is this fix supposed to have made it to release yet? Asking because I'm still seeing this issue with the current VPN release (I've got version 2.6.1, Google Play store isn't offering me an update beyond that currently).

I do see that the fix landed a month ago; I'm not sure what the release model is like for VPN so I'm not sure if it's coming in a future version-bump vs. if the patch was expected to be in the release at this point.

(No big rush; just want to be sure it's not a surprise that the bug is still present in release.)

@strseb
Copy link
Collaborator

strseb commented Dec 21, 2021

Hey! This should be part of the next release - so 2.7 :)

@dholbert
Copy link
Collaborator Author

OK great, thanks!

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.

2 participants