-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create status codes for session-related errors #191
Comments
Also |
I think there is still a race condition in case of token expiration, so it is possible to get expired token from storage. |
It is. My point was that those errors are pretty much the same? If it is expired and if it is not found, both indicate that client has to renew session token. Maybe it can be encoded as a detail, not the separate error code. I am not completely sure about that, though. |
I agree they are essentially the same, just that an error message in case of expired token can be confusing. Maybe user-facing software should just tell |
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
Currently there are 2 candidates https://github.com/nspcc-dev/neofs-node/blob/master/pkg/services/object/util/key.go#L13
The text was updated successfully, but these errors were encountered: