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.

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

Fixup settings.yaml and user-management.yaml #112

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

Conversation

WaldemarEnns
Copy link

@WaldemarEnns WaldemarEnns commented Jan 16, 2025

Description

Using openapi-generator-cli version 7.20.0, I get errors validating the specification with

openapi-generator-cli validate -i settings.yaml

and

openapi-generator-cli validate -i user-management.yaml

Output:

Validating spec (settings.yaml)
Errors:
	- attribute paths.'/api/v1/dns.resolve.txt'(post).requestBody.content with no media type is
	  unsupported
	- attribute paths.'/api/v1/dns.resolve.srv'(post).requestBody.content with no media type is
	  unsupported
	- attribute paths.'/api/v1/import.run'(post).requestBody.content with no media type is
	  unsupported

[error] Spec has 3 errors.

and

Validating spec (user-management.yaml)
Errors:
        - attribute paths.'/api/v1/users.2fa.disableEmail'(post).requestBody.content with no media
          type is unsupported
        - attribute paths.'/api/v1/users.2fa.enableEmail'(post).requestBody.content with no media
          type is unsupported
        - attribute paths.'/api/v1/users.logout'(post).requestBody.content with no media type is
          unsupported

[error] Spec has 3 errors.

This can be fixed by addressing the error-messages. In this case I checked the endpoints for their request-body and noticed they have none, so I fixed this by removing the requestBody annotation from the spec.

List of fixed endpoints:

Relates to #107

Could be removed safely, since the corresponding endpoints don't require any request body.
@CLAassistant
Copy link

CLAassistant commented Jan 16, 2025

CLA assistant check
All committers have signed the CLA.

Could be removed safely, since the corresponding endpoints don't require any request body.
@WaldemarEnns WaldemarEnns changed the title Fixup settings.yaml Fixup settings.yaml and user-management.yaml Jan 17, 2025
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.

2 participants