Skip to content
This repository has been archived by the owner on May 24, 2023. It is now read-only.

Document authorization #28

Merged
merged 1 commit into from
Mar 6, 2019
Merged

Conversation

MartinBasti
Copy link
Contributor

Documents how to use quay tokens with OMPS.

Fixes #27

Signed-off-by: Martin Bašti mbasti@redhat.com

README.md Outdated
@@ -95,18 +120,24 @@ Error messages have following format:
|-----------|------------------------|---------------------|
|400| OMPSUploadedFileError | Uploaded file didn't meet expectations (not a zip file, too big after unzip, corrupted zip file) |
|400| OMPSExpectedFileError | Expected file hasn't been attached |
|403| OMPSAuthorizationHeaderRequired| No `Authorization` header found in request|
|404| OMPSOrganizationNotFound | Requested organization is not configured on server side |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one was also removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! removed (both occurrences)

README.md Outdated
@@ -150,19 +181,23 @@ Error messages have following format:

| HTTP Code / `status` | `error` | Explanation |
|-----------|------------------------|---------------------|
|403| OMPSAuthorizationHeaderRequired| No `Authorization` header found in request|
|404| OMPSOrganizationNotFound | Requested organization is not configured on server side |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

Documents how to use quay tokens with OMPS.

Fixes release-engineering#27

Signed-off-by: Martin Bašti <mbasti@redhat.com>
@csomh csomh merged commit 6dbd055 into release-engineering:master Mar 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants