-
-
Notifications
You must be signed in to change notification settings - Fork 344
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
Google Play Services authorization prompt in v74 #281
Comments
This is likely due to new code in v74 using Play integrations; unfortunately the patches that I am using to evict this functionality are orphaned, so I would have now to do some work to find out why it's happening. |
I haven't seen this notification yet. When does it show up? |
This could be a regression (reference: https://forum.xda-developers.com/showpost.php?p=76242418&postcount=652) @ouzowtf you need to terminate the app or reboot to see it. |
Fixed in @dadosch please report back if it is not fixed. |
I can confirm that it isn't fixed in 74.0.3729.86. |
Ok, thanks for reporting; something must have changed in the code or in the manifest that makes this prompt happen again. No idea about what though. |
You can disable this notification in bromite's notification settings |
Is this problem still present with |
Yes. |
Just tested 74.0.3729.141 and login still isn't working. |
@AlternativeZero that should be discussed in a separate issue. This issue here is "Google Play Services authorization prompt in v74" |
I cannot reproduce this issue; can somebody please verify with |
The issue still exists in the latest version, |
@BugReporter2 you are experiencing this issue with the Bromite browser, not the SystemWebView, correct? What are the steps to trigger it? |
Yes, with the Bromite browser.
Just normal browsing. There is no special step to trigger it. Start browsing websites, so that prompt will come in a certain amount of time. Sometimes the prompt comes almost immediately when you open Bromite and start browsing and sometimes it can take several minutes. |
|
Hi @csagan5, can you supply a reference to those patches? |
@allenluce the patches have been included for about a year now, they come from this abandoned project: https://gitlab.com/thermatk/Unobtainium You can find them under https://github.com/bromite/bromite/tree/master/build/patches as This new authorization request is due to some new code from upstream and it's not easy to find out where it is; I also cannot reproduce it. |
Try visiting https://www.tomshardware.com/. |
@thestinger I am still unable to reproduce this with real devices. Can this be reproduced with the Android Virtual Device or it is related to running a specific ROM? |
It occurs for me on AOSP on any of the officially supported devices. I think there's a good chance you could reproduce it in the emulator. |
Also, worth noting that this does also occur in Brave too: brave/browser-android-tabs#1639, so the idea of getting a fix from there isn't viable. |
Another page which quickly triggers this issue is https://1.1.1.1/help |
@thestinger I think it does not appear on my physical devices because it might have been granted in the past already. Thanks for the explanation and reference, it makes sense. So it's an upstream bug and there appears to be little incentive from upstream to fix it in a timely manner. |
Has anybody found/created the upstream issue in Chromium issue tracker? I suggest addressing it upstream, they might find more easily where this comes from. This would be addressed radically if no play services were used in Bromite, but it's not currently being worked on. Closing it as the fix will come from upstream (if ever). |
I don't think there's an issue open yet. I wanted to identify the reason for the issue before filing it. They're likely to fix it very quickly if they're pointed towards the right place, but otherwise I expect they will treat it as a very low priority unless they can quickly identify the problem. Since https://1.1.1.1/help can trigger it without needing much code, someone should work on narrowing down the specific issue and then it should be fairly easy to find the right place to fix this. |
Will be fixed in next release |
Can someone please test whether this problem is resolved with latest version |
Fixed on my side. I can't seem to trigger it on sites that usually triggers this. Thanks guys. |
Bromite version
Version:
74.0.3729.80
Arch:
arm64
Android version:
7.1.2
Is the bug reproducible with latest version?
Already using latest version available from F-Droid Repo.
Can the bug be reproduced with corresponding Chromium version?
Not tested, but Chromium doesn't claim to be privacy-enhanced.
Is the bug a crash?
no
Describe the bug
A notification appears after the latest update saying that Bromite needs Google Play Services. (see screenshot)
Expected behavior
Bromite should not depend on Google Play Services
Screenshots
![Screenshot_20190413-162835](https://user-images.githubusercontent.com/12801390/56082644-be7c1e80-5e0a-11e9-969f-9c1bf9b40be2.png)
The text was updated successfully, but these errors were encountered: