Skip to content

Add ability to specify root security and securitySchemes components (Code style fixes)#26

Open
flofloflo wants to merge 4 commits intoswisnl:masterfrom
flofloflo:feature/security-scheme-builder
Open

Add ability to specify root security and securitySchemes components (Code style fixes)#26
flofloflo wants to merge 4 commits intoswisnl:masterfrom
flofloflo:feature/security-scheme-builder

Conversation

@flofloflo
Copy link
Copy Markdown

I've used the stale PR #16 as base and did the last bit of code style fixes to make it ready for a merge. Style checks and tests are passing. I was also able to verify the functionality in a manual test as well.

If more changes are needed, I'm happy to help to get this ready for master.

Description

This PR adds the ability to specify securitySchemes and the root level security array from the config file

Motivation and context

Needed for our docs and in the project's TODO list

How has this been tested?

Added tests for generation, locally tested with and without config

Screenshots (if appropriate)

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

  • I have read the CONTRIBUTING document.
  • My pull request addresses exactly one patch/feature.
  • I have created a branch for this patch/feature.
  • Each individual commit in the pull request is meaningful.
  • I have added tests to cover my changes.
  • If my change requires a change to the documentation, I have updated it accordingly.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

@JaZo JaZo requested a review from Rocksheep October 27, 2025 08:48
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.

1 participant