Skip to content

Conversation

@mickenordin
Copy link
Member

The tokenEndPoint does not pass validation:

spec.yaml: Validation Error: Path parameter 'tokenEndPoint' for 'post' operation in '/{tokenEndPoint}' was not resolved (using openapi-spec-validator)

This is an attempt to fix that by removing the endpoint and adding the information under the discovery instead, since this endpoint is discovered rather than fixed, I think this is how it should be done.

The tokenEndPoint does not pass validation:

spec.yaml: Validation Error: Path parameter 'tokenEndPoint' for 'post'
operation in '/{tokenEndPoint}' was not resolved (using
openapi-spec-validator)

This is an attempt to fix that by removing the endpoint and adding the
information under the discovery instead, since this endpoint is
discovered rather than fixed, I _think_ this is how it should be done.
@glpatcern
Copy link
Member

Interestingly, the CI does not fail. How did you see the failed validation?

And maybe you could remove the extra empty lines?

@mickenordin
Copy link
Member Author

mickenordin commented Dec 10, 2025

Interestingly, the CI does not fail. How did you see the failed validation?

I use the tool openapi-spec-validator (pipx install openapi-spec-validator)

And maybe you could remove the extra empty lines?

Certainly!

And also remove link which on second thought maybe should not be
there
Copy link
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

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

Looks good, we could integrate the openapi-spec-validator tool in the CI then

@mickenordin mickenordin merged commit 8cf8da1 into develop Dec 11, 2025
4 checks passed
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