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

App Crashes On Email Verification (Sign Up) #6855

Closed
ericdecanini opened this issue Aug 17, 2022 · 1 comment · Fixed by #6862
Closed

App Crashes On Email Verification (Sign Up) #6855

ericdecanini opened this issue Aug 17, 2022 · 1 comment · Fixed by #6862
Labels
O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Crash Z-FTUE Issue is relevant to the first time use project or experience

Comments

@ericdecanini
Copy link
Contributor

ericdecanini commented Aug 17, 2022

During a community testing session we found that during sign up that upon verifying your email, when reaching the login captcha fragment, the app crashes.

Rageshake:
https://github.com/matrix-org/element-android-rageshakes/issues/41896

I tried reproducing this issue myself (on matrix.org) but to no avail. It's possible that this is dependent on the homeserver.

What this meant for the user however is they could not sign up at all

@ericdecanini ericdecanini added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-FTUE Issue is relevant to the first time use project or experience labels Aug 17, 2022
@ericdecanini ericdecanini added S-Critical Prevents work, causes data loss and/or has no workaround O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Aug 17, 2022
@ouchadam
Copy link
Contributor

ouchadam commented Aug 17, 2022

going to downgrade the severity of this as it's caused by the device not having a webview installed

Caused by: android.util.AndroidRuntimeException: android.util.AndroidRuntimeException: android.webkit.WebViewFactory$MissingWebViewPackageException: Failed to load WebView provider: No WebView installed
	at android.webkit.WebViewFactory.getProvider(WebViewFactory.java:363)
	at android.webkit.WebView.getFactory(WebView.java:2676)
	at android.webkit.WebView.ensureProviderCreated(WebView.java:2670)
	at android.webkit.WebView.setOverScrollMode(WebView.java:2746)
	at android.view.View.<init>(View.java:5357)
	at android.view.View.<init>(View.java:5513)
	at android.view.ViewGroup.<init>(ViewGroup.java:747)
	at android.widget.AbsoluteLayout.<init>(AbsoluteLayout.java:56)
	at android.webkit.WebView.<init>(WebView.java:453)
	at android.webkit.WebView.<init>(WebView.java:395)
	at android.webkit.WebView.<init>(WebView.java:377)
	at android.webkit.WebView.<init>(WebView.java:364)
	... 32 more
Caused by: android.util.AndroidRuntimeException: android.webkit.WebViewFactory$MissingWebViewPackageException: Failed to load WebView provider: No WebView installed
	at android.webkit.WebViewFactory.getProviderClass(WebViewFactory.java:532)
	at android.webkit.WebViewFactory.getProvider(WebViewFactory.java:348)
	... 43 more
Caused by: android.webkit.WebViewFactory$MissingWebViewPackageException: Failed to load WebView provider: No WebView installed
	at android.webkit.WebViewFactory.getWebViewContextAndSetProvider(WebViewFactory.java:428)
	at android.webkit.WebViewFactory.getProviderClass(WebViewFactory.java:493)
	... 44 more

my suggestion would be to catch and inform the user about their missing WebView/Browser


https://stackoverflow.com/questions/43917214/android-how-to-check-if-webview-is-available/43925989#43925989

according to SO the webview feature check isn't reliable as a user may still disable the webiew/browser, the workaround is to try and instantiate the CookieManager and catch any exceptions

val hasWebView: Boolean = kotlin.runCatching { CookieManager.getInstance() }.isSuccess

@ouchadam ouchadam added S-Major Severely degrades major functionality or product features, with no satisfactory workaround and removed S-Critical Prevents work, causes data loss and/or has no workaround labels Aug 17, 2022
@ouchadam ouchadam mentioned this issue Aug 17, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Crash Z-FTUE Issue is relevant to the first time use project or experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants