What about sessionStorage?

[jsmunroe]

I had to fork this repo. Here is my fork. My team is using sessionStorage instead of localStorage because we want the data to live only for the duration of the user's session.

Both of these types are Storage types and so I generalized your SessionLocalStorage class into a SessionStorage class that takes a Storage object in its constructor. This way the sessionLocalStorage instance is instantiated like this...

const secureLocalStorage = new SecureStorage(localStorage);

I left secureLocalStorage as the default export to avoid breaking changes and added secureSessionStorage as an auxiliary export. This instance is instantiated as such...

const secureSessionStorage = new SecureStorage(sessionStorage);

sessionStorage is supported in 95.91% of browsers and adds a lot of flexibility when it comes to life spans of client-side data. Feel free to incorporate this change into your own repository, just let me know when you do so I can move my project back over to your package.

Thanks for the great code. Much luck.

[sushinpv]

@jsmunroe Thanks a lot for your contribution, Please give me sometime to go through your code and run a test against our usecases!

In the meantime, You can create a PR to the development,

Thank you for your contribution

[jsmunroe]

Thank you for your quick response and your willingness to look into this commit.

I think I did pull request the changes. I'm not sure. This is my first time collaborating on github. Let me know if I need to do something other than what I think I did. :)

[sushinpv]

Hi @jsmunroe We are testing your PR, once we have any updates we will let you know!

Thanks for waiting

[Anon0x19]

Hi there @sushinpv, I noticed the PR you raised hasn't been merged yet, and I'm not sure if it's been discontinued. I thought it might be good to mention it here.

I've raised a PR to implement session storage. I used the PR you both raised as guidance when implementing it and successfully tested it with one of my projects.

I hope we can get session storage support merged soon, as it would be a very useful feature.

Best regards,
Marc