Fix android clearCookies method

Cap-go / capacitor-inappbrowser

Capacitor plugin in app browser with urlChangeEvent

MIT License

57 stars 37 forks source link

Fix android clearCookies method #134

Closed karen1au closed 1 month ago

karen1au commented 1 month ago

Existing clearCookies method is not clearing/overwriting cookies that are set by the browser. Since the method is meant to clear all cookies instead of targeting specific cookie pairs anyways, this PR updates the method to call the removeAllCookies method instead.
Also updated the call to removeSessionCookie (now deprecated) to use removeSessionCookies instead

karen1au commented 1 month ago

@riderx We found this bug when testing on Android, and this fix seems to work, could you review when you get a chance and see if this is the proper fix as we do need it? TIA 🙇