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

590 roles in security schemes #615

Merged

Conversation

benjamin-confino
Copy link
Contributor

Resolves #590

No changes to smallrye were needed to pass the new tests.

Depends on the delivery of #613 - marking this as a draft until 613 is in.

@benjamin-confino benjamin-confino marked this pull request as draft May 14, 2024 12:04
@eclipse-microprofile-bot
Copy link
Contributor

Can one of the admins verify this patch?

@benjamin-confino benjamin-confino force-pushed the 590-roles-in-security-schemes branch from c68957f to d269097 Compare May 14, 2024 12:30
@benjamin-confino benjamin-confino force-pushed the 590-roles-in-security-schemes branch 2 times, most recently from 7867405 to 35e81b8 Compare May 14, 2024 14:56
@benjamin-confino benjamin-confino force-pushed the 590-roles-in-security-schemes branch 2 times, most recently from aead3cb to 6de4338 Compare May 15, 2024 10:53
@benjamin-confino benjamin-confino force-pushed the 590-roles-in-security-schemes branch from 6de4338 to a801f8c Compare May 15, 2024 12:53
@Azquelt Azquelt force-pushed the 590-roles-in-security-schemes branch from a801f8c to 4be1c82 Compare May 24, 2024 16:28
@Azquelt Azquelt force-pushed the 590-roles-in-security-schemes branch from 4be1c82 to 9e9289f Compare May 24, 2024 16:43
@Azquelt Azquelt marked this pull request as ready for review May 24, 2024 16:45
@Azquelt
Copy link
Member

Azquelt commented May 24, 2024

@eclipse-microprofile-bot test this please

@Azquelt Azquelt requested review from MikeEdgar May 24, 2024 16:59
@Azquelt Azquelt merged commit 84260d2 into microprofile:main May 29, 2024
4 checks passed
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.

[OAS 3.1.0] All security schemes may now define required roles
3 participants