You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by GitBox <gi...@apache.org> on 2020/07/01 09:45:06 UTC
[GitHub] [cordova-plugin-inappbrowser] chenheng0910 opened a new issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
chenheng0910 opened a new issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744
# Bug Report
## Problem
Hi there
I am working for a hybrid application using IONIC v3.
I use the in-app-browser to do a third-party login to Microsoft, and also to achieve the use fingerprint or facial print login.
And Android worked, but IOS failed .
### What is expected to happen?
success login both android and ios
### What does actually happen?
Android worked, but IOS failed .
## Information
<!-- Include all relevant information that might help understand and reproduce the problem -->
### Command or Code
<!-- What command or code is needed to reproduce the problem? -->
### Environment, Platform, Device
<!-- In what environment, on what platform or on which device are you experiencing the issue? -->
Platform: IOS
### Version information
<!--
What are relevant versions you are using?
For example:
Cordova: Cordova CLI, Cordova Platforms, Cordova Plugins
Other Frameworks: Ionic Framework and CLI version
Operating System, Android Studio, Xcode etc.
-->
ionic: 4.2.0
angular/core: 5.2.11
cordova: 9.0.0
cordova-ios: 5.0.0
cordova-android: 8.1.0
node: 10.16.0
npm: 6.9.0
## Checklist
<!-- Please check the boxes by putting an x in the [ ] like so: [x] -->
- [x] I searched for existing GitHub issues
- [x] I updated all Cordova tooling to most recent version
- [x] I included all the necessary information above
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] NiklasMerz commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652402040
iOS 14 will introduce FaceID and TouchID. Although I did no test it yet.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] NiklasMerz commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652817848
Version 4.0.0 should not have the file ` CDVUIInAppVrowser.m`. This was the UIWebView implementation that got removed in this version.
Cookies are indeed a difficult topic in inappbrowser and WKWebView but I don't understand why this should enable Fingperprint authentication that is only coming in the next major version of iOS.
Could you please provide more details? Possibly a video would help us to understand this.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] breautek commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
breautek commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652598332
Agreed. Closing because this isn't a bug. Perhaps in the future it will be a feature request, if something needs to be enabled in the WKWebView for iOS 14+
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] timbru31 commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
timbru31 commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652345500
Does the login works in plain Safari on iOS? (my guess is no)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] chenheng0910 commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
chenheng0910 commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652858578
@NiklasMerz
Sure, i will upload test video and my application's login log both android and ios.
But i don't know where should i upload these info that you could see it.
Maybe Email is ok if you don't mind
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] chenheng0910 commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
chenheng0910 commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652875064
IOS login log:
<img width="1294" alt="IOS-log" src="https://user-images.githubusercontent.com/18900304/86336483-80bf1c00-bc82-11ea-80e1-234933743ce9.png">
Android login log:
Here you can see ios return url with an error:
`A silent sign-in request was sent but no user is signed in.`
But android return url with a `code`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] NiklasMerz commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652865873
If you don't mind uploading it publicly please upload it to this issue or create a sample application to show the issue.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] chenheng0910 commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
chenheng0910 commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652732328
@breautek
I change in-app-browser plugin from 4.0.0 to 3.2.0 and add the code as follow in `CDVUIInAppVrowser.m` file in line 74, and IOS face login worked
`NSHTTPCookie *cookie;`
`NSHTTPCookieStorage *storage = [NSHTTPCookieStorage sharedHTTPCookieStorage];`
`for (cookie in [storage cookies])`
` {`
`NSDate *expiresDate = [NSDate dateWithTimeIntervalSinceNow:3600*24*30*12];`
`NSArray *newCookies = [NSArray arrayWithObjects:cookie.name, cookie.value, expiresDate, cookie.domain, cookie.path, nil];`
` if(newCookies){`
` NSMutableDictionary *cookieProperties = [NSMutableDictionary dictionary];`
`[cookieProperties setObject:[newCookies objectAtIndex:0] forKey:NSHTTPCookieName];`
`[cookieProperties setObject:[newCookies objectAtIndex:1] forKey:NSHTTPCookieValue];`
` [cookieProperties setObject:[newCookies objectAtIndex:2] forKey:NSHTTPCookieExpires];`
`[cookieProperties setObject:[newCookies objectAtIndex:3] forKey:NSHTTPCookieDomain];`
` [cookieProperties setObject:[newCookies objectAtIndex:4] forKey:NSHTTPCookiePath];`
` NSHTTPCookie *newCookie = [NSHTTPCookie cookieWithProperties:cookieProperties];`
`[[NSHTTPCookieStorage sharedHTTPCookieStorage] setCookie:newCookie];`
` }`
` }`
We think this problem is related to cookies. I'd like to know what you think？
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] timbru31 edited a comment on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
timbru31 edited a comment on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652345500
Does the login work in plain Safari on iOS? (my guess is no)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-plugin-inappbrowser] timbru31 commented on issue #744: IOS: Third part login use fingerprint login / face login, IOS failed and display a white screen.
Posted by GitBox <gi...@apache.org>.
timbru31 commented on issue #744:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/744#issuecomment-652408617
Yes:
> Added a Web Authentication platform authenticator using Face ID or Touch ID, depending on which capability is present.
(https://developer.apple.com/documentation/safari-release-notes/safari-14-beta-release-notes), but I'll doubt they a) backport it to iOS 13 and b) that this issue is invalid since it's a Safari issue we can't workaround.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org