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

Crash when default browser doesn't support Custom Tabs #422

Closed
serrq opened this issue Jun 20, 2024 · 16 comments · Fixed by #471
Closed

Crash when default browser doesn't support Custom Tabs #422

serrq opened this issue Jun 20, 2024 · 16 comments · Fixed by #471
Labels
bug Something isn't working

Comments

@serrq
Copy link

serrq commented Jun 20, 2024

Android 14
Leon 42 (fdroid)
OnePlus CPH2399

1000113892

This crash happen after I pushed on "open URLs in a custom tab..." in settings

@svenjacobs
Copy link
Owner

I currently only find version 41 on F-Droid. However I cannot reproduce this crash with 41 from F-Droid or 42 from Play Store. Please clear the app data and try if it helps. If not please provide a crashlog (Android logcat) if possible.

@svenjacobs svenjacobs added the question Further information is requested label Jun 20, 2024
@serrq
Copy link
Author

serrq commented Jun 20, 2024

Wiping app data solve the "launch" issue, but not the bug itself.

1000113894

@serrq
Copy link
Author

serrq commented Jun 20, 2024

Try this move and then update again Droid-ify or something-ify... (to find 42)

1000113897

@svenjacobs
Copy link
Owner

I installed 42 via Droid-ify. I only used the official F-Droid client before. Still no crash.
Please provide a crashlog or I have no idea what's going on.

@serrq
Copy link
Author

serrq commented Jun 20, 2024

I have no idea about how to building a crash log. I am not a developer. Sorry.

@svenjacobs
Copy link
Owner

What browser is installed on your device?

@serrq
Copy link
Author

serrq commented Jun 20, 2024

I have three browsers: DuckDuck, Mull, Fennec. All taken from droid-ify.

Okay, the crash occurs because there are three available options...

Could be so?

@svenjacobs
Copy link
Owner

Do you know if any of those browsers support Custom Tabs? It is a functionality that browsers must implement to support. Could you maybe as a test install the regular Firefox and see if it still crashes? I assume it has something to do with this code.

@serrq
Copy link
Author

serrq commented Jun 21, 2024

Yes, with the official "Firefox for Android" browser Leon doesn't crash, but only if Firefox is set as default browser. Having it installed is not enough, it needs to be set as default.

So what do you will do? Make an exception?

@svenjacobs
Copy link
Owner

I probably need to add a check if the current default browser even supports Custom Tabs.

@svenjacobs svenjacobs changed the title No longer able to launch Leon due a fatal (perpetual) crash Crash when default browser doesn't support Custom Tabs Jun 21, 2024
@svenjacobs svenjacobs added bug Something isn't working and removed question Further information is requested labels Jun 21, 2024
@serrq
Copy link
Author

serrq commented Jun 21, 2024

Ehm... default browser is Leon in my device...

1000113909

@svenjacobs
Copy link
Owner

Hm, that's interesting. Maybe having Leon as the default browser means Custom Tabs are not possible.

@serrq
Copy link
Author

serrq commented Aug 23, 2024

Any updates on this issue?

@svenjacobs
Copy link
Owner

@serrq I'm sorry, but I can't reproduce this crash. Even when Leon is set as the default browser and custom tabs are enabled, the app does not crash. Nothing happens when I click "open" because Leon as default browser can't handle custom tabs.

Can you please try with the latest version? Also please install via official F-Droid app, just to be sure.

Screenshot_20240921_161411

@svenjacobs svenjacobs added the question Further information is requested label Sep 21, 2024
@svenjacobs
Copy link
Owner

Forget what I said. I can reproduce this crash when restarting Leon. A fix is incoming.

@svenjacobs svenjacobs removed the question Further information is requested label Sep 21, 2024
@svenjacobs
Copy link
Owner

The fix just addresses the crash.

However, the situation where Leon is the default browser and thus can't open links from within Leon (because it's not a real browser) needs more attention. This also relates to #372.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants