Should useWebKit be default?

Question

Should we default useWebKit to true as the default for all static methods? (e.g. CookieManager.clearAll(true)) Given that react-native-webview from the community has now removed UIWebView entirely (link), it seems it would make sense to default to using WKWebView (aka useWebKit).

This would be a breaking chance, so despite 4.0 just being release it would probably to wait until 5.0 to be released.

Thoughts?

[davidgovea]

I had the similar idea, but after looking into it, the “default” mode (NSHTTPCookieStorage) is used for react-native’s network stack (i.e. fetch(), etc)

That would make changing the default quite a large change for anyone relying on that behavior (as opposed to just webview cookie management). Not sure if that is a common use-case, but I suppose the change would be OK if it is rare.

I actually need to manage the cookies on “both sides” in iOS, so I’ve been using hacks like:

for (const useWebkit of [false, true]) {
  // Call both non-webkit and webkit variants on iOS, but only once on Android
  if (!useWebkit || Platform.OS === 'ios') {
    CookieManager.set(url, cookie, useWebkit);
  }
}

To me, it would be nice to auto-sync the cookies between the two stores on iOS – but again, not sure if my usecase is common.

[safaiyeh]

Sweet ill take this up as a documentation task