-
Notifications
You must be signed in to change notification settings - Fork 46
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
Use "app link" for redirect #129
Comments
Seems only supported Android >= 6. I think currently we require Android >= 5 for the app. Is it okay to update the requirements to Android >= 6? Android < 6 is no longer supported in any way by Google. |
I did manage to get it to work at some point, but now it is broken again. No idea what is going on, it won't return to the correct place in the app, i.e. it will go back to the "add provider" page and not to the main screen. It is strange that you need to override the manifest with a "scheme" but you can't specify app links directly with this method with AppAuth so you DO have to modify the manifest of the app itself as well, so it seems there are now two ways to specify it. Ugly! |
What is your idea about this? Is this feasible? |
I think users would prefer compatibility with older devices with having the drawback of a less secure redirect. If we use custom tabs, then the redirect goes back to the app immediately, so this is only the case with the default browser AFAIK.
That should be easily fixable, we can detect if the app is opened from a URI scheme, and then open the correct screen.
I'm not entirely sure I understand what you mean, but with adding the scheme under an activity in the Manifest, you tell the system that the specific activity can be opened via that URL scheme you just added. |
For security software it makes sense to only support the versions of Android that are still supported and receive security updates... Not sure what is the lowest version that still receives (monthly) updates? Does e.g. the Fairphone actually fix security issues after Google stops supporting an Android version? Not sure what makes sense here...
It would be nice to implement this (using the HTTPS URL) because then we can disable the "authorize" dialog for the Android app! If this is not a (big) problem. then I guess it makes sense to go for it! :) |
This is manufacturer-dependent, but it is usually 2 years after release of the device.
Probably not.
The problem with this is, it only works with https redirect URIs. Right now our URI is |
https://github.com/eduvpn/vpn-user-portal/blob/v2/src/OAuthClientInfo.php#L42
Could this work? |
Looks good to me! |
If you can provide the contents of what needs to be in there exactly I can put it there :) |
(Oh, and there's also one for the Let's Connect! build...) |
Will do on Monday! |
@fkooman for this file I need the certificate fingerprint of the release keystore (see this documentation for how to retrieve it). |
Let's Connect!
We are still searching for the eduVPN keystore ;-) |
According to this line, it should be |
I also verified that this certificate/key was used to sign the last version available on https://app.letsconnect-vpn.org/ :) |
That's in a build-VM, not on my local machine, so I don't have it anymore there. Rogier for sure has it! |
Keystore type: JKS Your keystore contains 1 entry Alias name: eduvpn Extensions: #1: ObjectId: 2.5.29.14 Criticality=false |
Thanks! |
https://github.com/eduvpn/vpn-user-portal/blob/v2/src/OAuthClientInfo.php#L55
|
@fkooman attached you can find the two asset links files. Here's the documentation, if you would need it. |
the one for eduvpn.org should be online now! |
letsconnect-vpn.org is almost finished, just waiting for DNS. |
Let's Connect! link is also fixed now! |
App links are now available. I am not able to test if it really works because I don't have access to the release keystores, could you please test both Let's Connect and EduVPN @fkooman? Reproduction steps:
If you see the following selector, then app links is NOT working, or the app is not signed with the correct keys: |
From which build/commit/branch should this work? |
It was merged with #190, master should work. |
|
It seems only to work with Chrome as default browser and with Custom Tabs on. (Galaxy A6 with official Samsung rom, Android 9). Build 84 |
It seems that this will not be useful for our users, if it only works on Chrome (I tried it too with Fennec and Firefox, and they both don't work). |
As discussed, I have reverted the app links change. App links seems not to be ready for production, at least not for redirect URLs. |
The documentation seems to have improved a lot: https://developer.android.com/training/app-links/verify-site-associations |
Hello try to use my URL application website but i have javax.net.ssl.SSHHandshakeException java.security.cert.certpathValidatorException Trust hanchor for certication path not found. When i use my own certificat (or Let'sencrypt ) it's failed but with iPhone it's ok website also it's ok Please how to fix it. |
https://tools.ietf.org/html/rfc8252#section-7.2
It seems this is something we have to do, but how is this enforced? is this something that needs "domain validation" in the play store? Or how does that work?
The text was updated successfully, but these errors were encountered: