Skip to content

Spec: Include an invalid credentials error on all calls that get the credentials to use #127

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.

Sign up for GitHub

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

Jump to bottom
Closed
govint opened this issue Oct 23, 2017 · 1 comment
Closed

Spec: Include an invalid credentials error on all calls that get the credentials to use #127

govint opened this issue Oct 23, 2017 · 1 comment
Assignees
@jdef
Labels
priority/p2
Milestone
v1.0

Comments

@govint
Copy link

govint commented Oct 23, 2017

All calls to the plugin that get the credentials as an argument must include an error to handle invalid credentials.
@jdef
Copy link
Member

jdef commented Dec 19, 2017

error codes were rewritten in #115. is this sufficient, or is there still a need for changes here?

@saad-ali saad-ali added this to the v1.0 milestone Nov 2, 2018
This was referenced Nov 2, 2018
Closed
Closed
jdef pushed a commit to jdef/spec that referenced this issue Nov 3, 2018
spec: revise common error codes
c562ed1 
* consolidate explanation of UNIMPLEMENTED.
* add specification for PERMISSION_DENIED.
* add specification for UNAUTHENTICATED.

Fixes container-storage-interface#127
@jdef jdef mentioned this issue Nov 3, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdef jdef

Labels
priority/p2
Projects
None yet
Milestone
v1.0
Development

No branches or pull requests
3 participants
@jdef @saad-ali @govint