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

Fixes Android when default browser is not support CustomTabs #111

Merged
merged 7 commits into from
Jun 6, 2024

Conversation

kecson
Copy link

@kecson kecson commented May 27, 2024

fix: if the default browser is not support customtabs , after sign in , it cannot redirect to app.

@ThexXTURBOXx
Copy link
Owner

Thank you very much for your PR! From a first quick look, it seems to be fine.
I will test properly once I am back from my vacation (should be in about a week)!

Copy link
Owner

@ThexXTURBOXx ThexXTURBOXx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the late reply - I was on honeymoon.
I have a few changes/comments :)

Comment on lines +33 to +38
* must register in AndroidManifest.xml :
* <intent-filter>
* <action android:name="android.intent.action.SEND" />
* <category android:name="android.intent.category.DEFAULT" />
* <data android:mimeType="text/plain" />
*</intent-filter>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add proper documentation in the README maybe? For example, in the Setup/Android section?
If it is not mandatory to do this, please properly point that out

Copy link
Author

@kecson kecson Jun 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sometimes it may open a browser that cannot auto redirect back to app, and need to add this to receive sharing from the browser.

Comment on lines +45 to +49
<intent-filter>
<action android:name="android.intent.action.SEND" />
<category android:name="android.intent.category.DEFAULT" />
<data android:mimeType="text/plain" />
</intent-filter>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that this is needed if autoVerify="true". But is it really needed in the example?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the scheme is not http/https, so don't need add autoVerify="true".

flutter_web_auth_2/lib/src/options.dart Outdated Show resolved Hide resolved
@ThexXTURBOXx ThexXTURBOXx merged commit 963111a into ThexXTURBOXx:master Jun 6, 2024
1 of 2 checks passed
@ThexXTURBOXx
Copy link
Owner

ThexXTURBOXx commented Jun 6, 2024

Thank you very much for your contribution! I will fix the remaining issues upstream

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 this pull request may close these issues.

2 participants