Skip to content
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

swagger: request url is misconfigured #1280

Closed
Dentrax opened this issue Jun 2, 2021 · 1 comment · Fixed by #1282
Closed

swagger: request url is misconfigured #1280

Dentrax opened this issue Jun 2, 2021 · 1 comment · Fixed by #1282

Comments

@Dentrax
Copy link

Dentrax commented Jun 2, 2021

Description of Problem / Feature Request

I think request URL is misconfigured on the Swagger. Currently, it tries to send a [POST] request to: localhost:8082indexer

I tried on this commit: 45538e0

Expected Outcome

I guess, it should be: http://localhost:8082/indexer/api/v1/index_report

Actual Outcome

Screen Shot 2021-06-02 at 17 56 04
hdonnay added a commit to hdonnay/clair that referenced this issue Jun 2, 2021
@hdonnay
openapi: fix paths
182041d 
According to my read of the
[spec](https://swagger.io/specification/#pathsObject), the paths need to
be prefixed with slashes.

This should fix quay#1280.

Signed-off-by: Hank Donnay <hdonnay@redhat.com>
@hdonnay hdonnay mentioned this issue Jun 2, 2021
Merged
hdonnay added a commit to hdonnay/clair that referenced this issue Jun 2, 2021
@hdonnay
openapi: fix paths
eff6353 
According to my read of the
[spec](https://swagger.io/specification/#pathsObject), the paths need to
be prefixed with slashes.

This should fix quay#1280.

Signed-off-by: Hank Donnay <hdonnay@redhat.com>
@hdonnay
Copy link
Member

hdonnay commented Jun 2, 2021

I think this is fixed in the linked PR.

@hdonnay hdonnay closed this as completed in 5d03223 Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

openapi: fix paths hdonnay/clair
2 participants
@hdonnay @Dentrax