search

pichillilorenzo / flutter_inappwebview

A Flutter plugin that allows you to add an inline webview, to use a headless webview, and to open an in-app browser window.
https://inappwebview.dev
Apache License 2.0
3.24k stars 1.59k forks source link

Android setCookie does not work with uppercased Cookies #929

Closed ma-pe closed 3 years ago

ma-pe commented 3 years ago

Environment

Technology Version
Flutter version 2.2.3
Plugin version 5.3.2
Android version 9.0 (API Version 28)

Device information: Google Pixel 2 (Emulator)

Description

Expected behavior: Setting a Cookie 'LoginToken' results in the Cookie being present.

Current behavior: Setting a Cookie 'LoginToken' adds the Cookie without values. All parameters are null: value, expiration, secure, etc.

Steps to reproduce

await cookieManager.setCookie(
      url: Uri.parse('https://test.com/'),
      name: 'LoginToken',
      value: 'abc12',
      maxAge: 30 * 60 * 1000);

print(await cookieManager.getCookie(
        url: Uri.parse('https://test.com/'), name: 'LoginToken'));

=> The same code works on iOS as expected. => The same code works on Android if I use the Cookie name 'logintoken' instead of 'LoginToken'.

github-actions[bot] commented 3 years ago

👋 @ma-pe

NOTE: This comment is auto-generated.

Are you sure you have already searched for the same problem?

Some people open new issues but they didn't search for something similar or for the same issue. Please, search for it using the GitHub issue search box or on the official inappwebview.dev website, or, also, using Google, StackOverflow, etc. before posting a new one. You may already find an answer to your problem!

If this is really a new issue, then thank you for raising it. I will investigate it and get back to you as soon as possible. Please, make sure you have given me as much context as possible! Also, if you didn't already, post a code example that can replicate this issue.

In the meantime, you can already search for some possible solutions online! Because this plugin uses native WebView, you can search online for the same issue adding android WebView [MY ERROR HERE] or ios WKWebView [MY ERROR HERE] keywords.

Following these steps can save you, me, and other people a lot of time, thanks!

ma-pe commented 3 years ago

I got it to work, without changing anything but clearing existing cookies first. So I think while testing I created a corrupt cookie, that prevented the correct cookie from being set. => Closed.