mirror
/
FairEmail
mirror of https://github.com/M66B/FairEmail.git
1
0
Fork 0
Code Issues Releases Wiki Activity

SBrowser: do not use customtabs for OAuth

This commit is contained in:
M66B 2022-11-04 08:54:05 +01:00
parent 677bb4e0fa
commit fd60e82d76
1 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
app/src/main/java/eu/faircode/email/FragmentOAuth.java
View File

@ -389,15 +389,24 @@ public class FragmentOAuth extends FragmentBase {
Browsers.SBrowser.SIGNATURE_SET,
false,
VersionRange.atMost("5.3"));
final BrowserMatcher SBROWSER_TAB = new VersionedBrowserMatcher(
Browsers.SBrowser.PACKAGE_NAME,
Browsers.SBrowser.SIGNATURE_SET,
true,
VersionRange.atMost("5.3"));
@Override
public boolean matches(@NonNull BrowserDescriptor descriptor) {
boolean accept = !(SBROWSER.matches(descriptor) || SBROWSER_TAB.matches(descriptor));
/*
java.lang.IllegalStateException: Too many bind requests(999+) for service Intent { act=android.support.customtabs.action.CustomTabsService pkg=com.sec.android.app.sbrowser }
at android.app.ContextImpl.bindServiceCommon(ContextImpl.java:2115)
at android.app.ContextImpl.bindService(ContextImpl.java:2024)
at android.content.ContextWrapper.bindService(ContextWrapper.java:870)
at android.content.ContextWrapper.bindService(ContextWrapper.java:870)
at androidx.browser.customtabs.CustomTabsClient.bindCustomTabsService(CustomTabsClient:80)
at net.openid.appauth.browser.CustomTabManager.bind(CustomTabManager:95)
at net.openid.appauth.AuthorizationService.<init>(AuthorizationService:117)
at net.openid.appauth.AuthorizationService.<init>(AuthorizationService:95)
at eu.faircode.email.FragmentOAuth.onAuthorize(FragmentOAuth:412)
*/
boolean accept = !(SBROWSER.matches(descriptor) ||
(descriptor.useCustomTab &&
Browsers.SBrowser.PACKAGE_NAME.equals(descriptor.packageName)));
EntityLog.log(context,
"Browser=" + descriptor.packageName +
":" + descriptor.version +