Skip to content

Allow missing security schemes#915

Open
velaskuez wants to merge 1 commit into
OpenAPITools:masterfrom
velaskuez:allow-missing-security-schemes
Open

Allow missing security schemes#915
velaskuez wants to merge 1 commit into
OpenAPITools:masterfrom
velaskuez:allow-missing-security-schemes

Conversation

@velaskuez

@velaskuez velaskuez commented Jul 23, 2026

Copy link
Copy Markdown

Fixes #737

A missing components.securitySchemes is valid OpenAPI, and as such should be accepted for diffing, rather than throwing an exception as it's currently doing.


Summary by cubic

Allow diffing OpenAPI documents that omit components.securitySchemes, treating them as valid and unchanged instead of throwing errors. Fixes #737.

  • Bug Fixes
    • Added null-safe handling for missing components.securitySchemes in security diff logic; treat absence as a no-op.
    • Updated tests to assert unchanged for specs without security schemes; still throw when a security requirement references a non-existent scheme.

Written for commit 8729ac4. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 4 files

Re-trigger cubic

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.

Missing #/components/securitySchemes section leads to IllegalArgumentException

1 participant