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/09 07:29:23 UTC
[GitHub] [cordova-plugin-inappbrowser] xinhuadian opened a new issue #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
xinhuadian opened a new issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751
# 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] -->
- [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] XandiKarlen edited a comment on issue #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
XandiKarlen edited a comment on issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751#issuecomment-733695616
@NiklasMerz We also had a problem just recently with managing the session cookie between different instances of the InAppBrowser. What we did was changing up the import in the `CDVWKInAppBrowser.m` to use
`../../../CordovaLib/Classes/Private/Plugins/CDVWebViewEngine/CDVWebViewProcessPoolFactory.h` instead of the one currently used and also change the instance of the `sharedFactory` object.
Maybe this will help
----------------------------------------------------------------
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 #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751#issuecomment-733682717
I am now affected by this problem. I am working on a pull request right now.
Seems like the new `CDVWebViewProcessPoolFactory` is not public and the imports won't work the same. I just need to figure out how to solve this and this may require an updated version of cordova-ios.
----------------------------------------------------------------
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] mitsoe commented on issue #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
mitsoe commented on issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751#issuecomment-693209459
Hey @NiklasMerz , any timeline for the 4.0.1 release?
----------------------------------------------------------------
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 #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751#issuecomment-695368302
Sorry we don't give any release dates. This issue is not addressed yet. Pull requests welcome.
----------------------------------------------------------------
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 #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751#issuecomment-733717357
I found a fix but this also requires a fix in cordova-ios. Let's see how the reviews go there.
----------------------------------------------------------------
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] XandiKarlen commented on issue #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
XandiKarlen commented on issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751#issuecomment-733695616
@NiklasMerz We also had a problem just recently with managing the session cookie between different instances of the InAppBrowser. What we did was changing up the import in the `CDVWKInAppBrowser.m` to use
`../../../CordovaLib/Classes/Private/Plugins/CDVWebViewEngine/CDVWebViewProcessPoolFactory.h` instead of the one currently used and also change the instance of the `sharedFactory`.
Maybe this will help
----------------------------------------------------------------
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 closed issue #751: WKWebview cookie sometimes can't be get after upgrade to Cordova 6.1.0
Posted by GitBox <gi...@apache.org>.
NiklasMerz closed issue #751:
URL: https://github.com/apache/cordova-plugin-inappbrowser/issues/751
----------------------------------------------------------------
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