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.

Sign up for GitHub

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

Jump to bottom

Remove unused security schemes #305

Merged

mlagally merged 1 commit into main from mlagally-patch-7

Oct 19, 2022

Contributor

mlagally commented Oct 19, 2022 •

edited by pr-preview bot

Loading

Reducing the list of security schemes and select the most widely used.


          Remove unused security schemes

5288d1a

mlagally requested review from benfrancis, egekorkan, sebastiankb, chachamimm, k-toumura and mryuichi

October 19, 2022 10:21

egekorkan approved these changes

View reviewed changes

Contributor Author

mlagally commented Oct 19, 2022

Arch call on Oct 19th:
Agree to merge, we will need to refine the contraints as per #221 in the future.

mlagally merged commit 7eb3fa0 into main

egekorkan mentioned this pull request

Security Schemes are too loose #221

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

egekorkan egekorkan approved these changes

benfrancis Awaiting requested review from benfrancis

sebastiankb Awaiting requested review from sebastiankb

chachamimm Awaiting requested review from chachamimm

k-toumura Awaiting requested review from k-toumura

mryuichi Awaiting requested review from mryuichi

Labels

None yet