block all cookies in safari breaks the hook

[bobaaaaa]

If you "block all cookies" in safari, this hook throws uncatched errors. It is not enough to surround the localStorage.getItem() call in a try-catch block. Even the localStorage access throws:

This happened also for https://github.com/astoilkov/use-session-storage-state

Safari (16.4) settings promt. (sorry for the german language)

[astoilkov]

I will fix this soon. Thanks!

[astoilkov]

This is now part of the latest release. Again, thanks for reporting!

I will migrate the change to use-session-storage-state as well soon.

[bobaaaaa]

Thank you. It works 👍. Will wait for use-session-storage-state because we are using both libs

[bobaaaaa]

@astoilkov should I create an issue on the https://github.com/astoilkov/use-session-storage-state repo?

[astoilkov]

Yeah, would probably be easier for you to keep track. Here I've done it: https://github.com/astoilkov/use-session-storage-state/issues/2.

Out of curiosity, where are you using both libraries?

[bobaaaaa]

Thanks 👍. My team is building t-online.de. It is one of the biggest german (online) news portals (300mio visits/month)

[astoilkov]

Wow. Is your organization open to sponsoring my work?

I'm not sure if you are familiar with the situation in open-source. It's the work of a lot of people that get little credit and money for the work they do. I think big organizations like yours are a way to improve that situation. This is a growing problem and the open-source community is trying to find a solution. This is why I'm experimenting with asking (inspiration: https://reactflow.dev/blog/asking-for-money-for-open-source/).

Thanks.

[bobaaaaa]

Sure. I will ask my boss.

[astoilkov]

Thank you!