Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

chore: improve validation of Security tag in swaggerparser#15660

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
defelmnq merged 5 commits intomainfrom15605-improve-swagger-parser
Nov 26, 2024

Conversation

defelmnq
Copy link
Contributor

Aims toresolve#15605

There's currently one option valid for the@Security tag in swaggerparser - which fails in the CI if we try to put any other value.

At least one of our endpoints does not acceptCoderSessionToken as an option for the authentication and so we need to add new possibilities in order to keep the documentation up-to-date.

In this PR , I addedProvisionerKey which is the way our provisioner daemon can authenticate to the backend - also modified a bit the code to simplify other options later.

@defelmnq
Copy link
ContributorAuthor

Merging this one after CI - onlyweekly-docs step is failing - which is not related to this PR but the tool currently used - which is why I voluntarily ignore it.

@dannykopping
Copy link
Contributor

Merging this one after CI - onlyweekly-docs step is failing - which is not related to this PR but the tool currently used - which is why I voluntarily ignore it.

Normally this step failing shouldn't be ignored, but since it's due to postgres' site being down (maybe they host on fly.io 🤔) it's fine 👍

@defelmnqdefelmnq merged commit8afb10e intomainNov 26, 2024
27 of 28 checks passed
@defelmnqdefelmnq deleted the 15605-improve-swagger-parser branchNovember 26, 2024 06:19
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@dannykoppingdannykoppingdannykopping approved these changes

@mafredrimafredriAwaiting requested review from mafredri

Assignees

@defelmnqdefelmnq

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add more Security options in swaggerparser
2 participants
@defelmnq@dannykopping
[8]ページ先頭
©2009-2025 Movatter.jp