react-native-cookies icon indicating copy to clipboard operation
react-native-cookies copied to clipboard

Added 'set' method for android

Open echo8795 opened this issue 6 years ago • 5 comments

adding the set cookie method for android by using react-native's ForwardingCookieHandler class

echo8795 avatar Nov 13 '18 07:11 echo8795

@joeferraro Please merge this pull request, if it has no problem. I need the 'set' method too.

caojianfeng avatar Dec 24 '18 09:12 caojianfeng

@joeferraro Please bump!

almirfilho avatar Mar 29 '19 23:03 almirfilho

@echo8795 , Thanks for the PR. I tried to use your master branch, the same code works well with react-native-webview on iOS when sharedCookiesEnabled is true.The cookie was accepted by login server, the login process bypassed.

However it still needs login on Android, do you have any knowledge about it?

Thanks again

wangghon avatar Jun 14 '19 13:06 wangghon

After the comments, I compared the cookie content to the one on iOS, they were different, then I found the issue of RN can only pass one cookie on Android.

After I apply the work around (https://facebook.github.io/react-native/docs/network.html#known-issues-with-fetch-and-cookie-based-authentication), unfortunately the login page still can't bypass

wangghon avatar Jun 15 '19 08:06 wangghon

I have forked the repo here: https://github.com/safaiyeh/react-native-cookie-store We can create issues and PRs there to continue the development of the project.

safaiyeh avatar Dec 11 '19 19:12 safaiyeh