Skip to content

add create index API as an async job #9

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pdowler
Copy link
Collaborator

@pdowler pdowler commented May 20, 2025

this proposed solution is for a parameter-style UWS job that fits the current DAL landscape

this proposed solution is for a parameter-style UWS job that fits the
current DAL landscape
Comment on lines 49 to 50
405:
$ref: ./vosi-std-responses.yaml#/not-implemented
Copy link
Member

Choose a reason for hiding this comment

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

405 is not likely to be an appropriate response here - it's only correct if this resource exists but does not support any POST requests (and would have to be accompanied by an Allow header giving a list of HTTP methods that are supported for that resource) - see RFC9110 sec 15.5.6.

Copy link
Collaborator Author

@pdowler pdowler May 21, 2025

Choose a reason for hiding this comment

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

Do you think that a 404 when probing for an optional endpoint is sufficient for the client?

This is a UWS job list endpoint so I agree that optional means it is either there or not, but technically the possibility of a 405 is really up to UWS. That's kind of the awkward thing about UWS and OpenAPI: the job description (params) have to be here but the response can come from the UWS component.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I guess the job list (GET) would also have to return a 404. Seems natural to me because a 1.1 TAP implementation would return a 404.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants