[FEATURE] Default to local storage

[chasefleming]

Issue to be solved

Because FCL uses session storage instead of local storage for storing the current user every time you open a new tab in your browser or your mobile browser you have to re-authenticate. This bumps some road bumps in the UX. I know this decision was originally made for security reasons, but after seeing the UX challenges I think it's time to revisit this topic and see if there is a way to make local storage the default option.

Suggest A Solution

No response

What are you currently working on that this is blocking?

No response

[bluesign]

I totally agree, last time I checked also storage strategies were not exported. Exporting them also can be very nice. So people can just set via config easily.

[nialexsan]

by design fcl storage can be set up to use custom implementation like this:

const isServerSide = () => typeof window === "undefined"

const getLocalStorage = () => {
  try {
    const LOCAL_STORAGE = {
      can: !isServerSide(),
      get: async key => JSON.parse(localStorage.getItem(key)),
      put: async (key, value) => localStorage.setItem(key, JSON.stringify(value)),
    }
    return LOCAL_STORAGE
  } catch (error) {
    return null
  }
}

...

config.put("fcl.storage.default", getLocalStorage())

[bluesign]

Is there a specific reason we don't expose them as alternatives?

‘config.put("fcl.storage.default",fcl.SESSION_STORAGE) ‘ etc

[chasefleming]

Yeah @bluesign I like that idea. We could just have what @nialexsan mentioned as options to set for the most common.