-
Notifications
You must be signed in to change notification settings - Fork 13
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
Be able to set cookies on the client #13
Comments
Thank you for your idea and kind words. This sounds like a really good. Unfortunately I probably won't be able to develop this myself soon, but I am happy to review and merge a pull request for this. You could have a look #12 as this could be somewhat similar. |
Thanks for your fast response, I will try to cover this in the upcoming weeks ( |
I am not entirely sure but this I think it should relatively straigthforward to implement a function in this plugin to add cookies in the cookie pool the proxy uses. |
Is your feature request related to a problem? Please describe.
If I am not mistaken (and sorry for the noise if I am),
cordova-plugin-webview-proxy/src/ios/WebviewProxy.m
Lines 50 to 59 in 0e5bfc9
document.cookie
) won't be used in any request. It would be nice to set cookies on the client as well.Describe the solution you'd like
Ideally, something like
window.WebviewProxy.setCookie()
with the same behavior asdocument.cookie
should do the job.Additional context
I am testing this in a Meteor/Cordova context where I had no luck with
app-bound-domains
. This issue explains the issue in conjunction with Meteor risetechnologies/cookieTest#2 (comment)Also thanks a lot @NiklasMerz for open sourcing your approach, IMHO this is a slim and neat way to tackle
ITP
issues! 🚀The text was updated successfully, but these errors were encountered: