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/07 15:14:12 UTC
[GitHub] [cordova-ios] xinhuadian opened a new issue #934: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
xinhuadian opened a new issue #934:
URL: https://github.com/apache/cordova-ios/issues/934
# Bug Report
## Problem
Try to get the cookie of inappbrowser. But sometimes can't get correct cookie.
This issue happen after upgrade cordova to Cordova ios 6.1.0. and inappbrowser upgrade to 4.0.0
### What is expected to happen?
Use blow code to get cookies in decidePolicyForNavigationAction of CDVWKInAppBrowser
WKHTTPCookieStore *wkWebViewCookieStore =[[[theWebView configuration] websiteDataStore] httpCookieStore];
[wkWebViewCookieStore getAllCookies:^(NSArray* wkcookies) {
for (NSHTTPCookie* cookie in wkcookies)
{
NSLog(@"wk cookie name is %@",cookie.name);
}
}];
### What does actually happen?
Some times the wkcookies can't get the correct cookie.
And this issue not always happen, but sometimes. in my test ,50% percentage happen.
## Information
<!-- Include all relevant information that might help understand and reproduce the problem -->
This issue not happen after downgrade the cordova ios to 5.1 and still use inappbrowser 4.0.0(after resolve some compile error).
### Command or Code
<!-- What command or code is needed to reproduce the problem? -->
Please see above
### Environment, Platform, Device
<!-- In what environment, on what platform or on which device are you experiencing the issue? -->
Cordova 9.0.0, Cordova ios 6.1.0, xcode 11.5.
Iphone simulator(13.5) and Iphone X (13.5)
### 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.
-->
## Checklist
<!-- Please check the boxes by putting an x in the [ ] like so: [x] -->
- [ ] I searched for existing GitHub issues
- [ ] I updated all Cordova tooling to most recent version
- [ ] 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-ios] xinhuadian commented on issue #934: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
xinhuadian commented on issue #934:
URL: https://github.com/apache/cordova-ios/issues/934#issuecomment-655912399
I just find the cause of this issue.
Inappbrowser 4.0.0 still use below code
#if __has_include("CDVWKProcessPoolFactory.h")
#import "CDVWKProcessPoolFactory.h"
#endif
CDVWKProcessPoolFactory should modify to CDVWebViewProcessPoolFactory
Wish next version can have this issue fixed.
----------------------------------------------------------------
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-ios] NiklasMerz commented on issue #934: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #934:
URL: https://github.com/apache/cordova-ios/issues/934#issuecomment-655954754
Thanks for reporting. Good find!
Sounds like an issue in inappbrowser. I am transfering this to the inappbrowser repo.
----------------------------------------------------------------
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