Identity sometimes asks for approval when decrypting even with access level 4 about identity HOT 9 CLOSED

@maebeam , any input on this? Would be good to know if this behaviour is expected, because it's pretty breaking right now for us.

from identity.

Sorry I missed this, for some reason my notifications were misconfigured.

Safari expires all javascript cookies after one week. If the cookie is expired we need the user to visit identity and click a button so we can store the cookie for another week. It sounds like we may need a special case where we use localStorage for Safari webviews embedded in apps

from identity.

Should be as simple as changing this function:

  mustUseStorageAccess(): boolean {
    return typeof document.hasStorageAccess === 'function' && !this.globalVars.webview;
  }

from identity.

Give that a try locally and either submit a PR or I can make the change for you

from identity.

I can give it a try locally, but if I understand correctly, I'll not be able to know if it's worked for a week? I think I'd prefer if you just made the change if that's ok, we've got actual real life users on our app now, so would be best if the fix got in as quickly as possible, and that'll probably happen faster with you making the changes.

I imagine after this fix is applied, user's will need to log out and back in to make sure identity stores things in the right place? If so, I'll need to put something into the app to make that happen when it needs to.

from identity.

Actually, I was able to verify locally that it works by changing the system time. I'll raise a PR now. Unfortunately it does seem that users will have to log out and back in again to avoid getting stuck, but I don't think that can be helped.

from identity.

I think this is probably resolved now, but I'd like to leave it open until I can verify that cookies are no longer expiring and locking users out if that's alright? Should know in a week or so.

from identity.

Sure, sounds good

from identity.

Closing since this seems resolved

from identity.