Supports ephemeralWebBrowserSession on iOS 13

[avenner]

Hello,

Actually, on iOS 12+, when you login, the cookie is kept until the app is killed.

Since iOS 13 it's possible to make session ephemeral.
With this option prefersEphemeralWebBrowserSession, the cookie is killed when tab browser is closed.

Pros :

• no need to logout manualy users (for account switch for example)
  Cons :
• user should reconnect each time the tab is presented.

It fix #76 , on iOS 13.

TODO : Improvement : make this parameter as an option

Best Regards

PR Checklist

• The PR title follows our guidelines: https://github.com/proyecto26/react-native-inappbrowser/blob/master/CONTRIBUTING.md#pull-request-process.
• There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
• All existing tests are passing
• Tests for the changes are included

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

[jdnichollsc]

Is it possible to add a flag from iOS options in order to enable/disable this? What do you think?
Sorry for the delay and thanks for your help! <3

[jorgexgb]

@avenner have you found an iOS option for this? I can't find anything we can put on Info.plist.

+1 for the PR

[jdnichollsc]

Guys, are you agree to merge this PR?

[avenner]

Sorry guys :

@jdnichollsc : very busy last time, I will try to add this as an option

Lasts commits was a mistake, will be cleaned soon, sorry for incovenience.

[jorgexgb]

@avenner no worries! thanks for the help.

[avenner]

@jorgexgb no need to add nothing in plist, need only to set the session as ephemeral ;)
But only working in iOS 13.

[avenner]

( mistake reverted, now looking to add this as an option... )

[avenner]

Done & tested.
Hope it will help !

[avenner]

(extra comma fixed)

[jorgexgb]

@avenner thanks!!!

[jorgexgb]

@avenner seems good! @jdnichollsc let us know if okay to merge

[avenner]

You are welcome @jorgexgb !

Please note that when this option is activated, there is no prompt before open auth ( "app" wants to use "example.com" ... )

[jorgexgb]

@avenner got it. That is totally fine!

[jdnichollsc]

Please fix the name of the property, using "ephemeralWebSession" instead of "animated"

[jdnichollsc]

Ohh and please include this new option here:

• https://github.com/proyecto26/react-native-inappbrowser/blob/master/index.d.ts#L15
• https://github.com/proyecto26/react-native-inappbrowser/blob/master/index.js#L28

Thanks for your help! 👍

[jdnichollsc]

Thanks for your contribution! <3

[avenner]

Big thanks to you @jdnichollsc and @jorgexgb !

[jorgexgb]

I really didn't do anything. @avenner @jdnichollsc you guys did everything! Thanks!!! 👍
I pulled the merge and seems to be working!

[jdnichollsc]

Ok, check the new version 3.4.0 👍

[avenner]

Nice !

[jdnichollsc]

Thanks to you! Sorry for the delay 😄

[avenner]

No problem :)