Skip to content

Conversation

@IGx89
Copy link
Contributor

@IGx89 IGx89 commented Oct 24, 2025

Fixes the security requirement not actually being added to the OpenAPI spec in the last code example in the "Use document transformers" section of https://learn.microsoft.com/en-us/aspnet/core/fundamentals/openapi/customize-openapi?view=aspnetcore-10.0#use-document-transformers

Every operation before this change:

"security": [{ }]

After:

"security": [{ "Bearer": [] }]

Internal previews

📄 File 🔗 Preview link
aspnetcore/fundamentals/openapi/customize-openapi.md Customize OpenAPI documents

Fixes the security requirement not actually being added to the OpenAPI spec.
@IGx89
Copy link
Contributor Author

IGx89 commented Oct 24, 2025

CC @xC0dex for review

@xC0dex
Copy link
Contributor

xC0dex commented Oct 24, 2025

You're so right! Thanks for the fix!

@guardrex guardrex requested a review from captainsafia October 25, 2025 10:15
@guardrex guardrex self-assigned this Oct 25, 2025
Copy link
Collaborator

@guardrex guardrex left a comment

Choose a reason for hiding this comment

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

Thanks @IGx89 ... Please also update the article date (ms.date in the metadata at the top) of ...

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/openapi/customize-openapi.md

... to make sure the build system pulls this change into the sample code shown.

@IGx89 IGx89 requested a review from guardrex October 29, 2025 13:29
Copy link
Collaborator

@guardrex guardrex left a comment

Choose a reason for hiding this comment

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

Thanks, @IGx89! ... Safia should be along shortly for a final signoff. In addition to the GH ping, I sent her an email, too. She's very busy these days! 🏃‍♀️

@guardrex
Copy link
Collaborator

guardrex commented Oct 30, 2025

No response ... let's go ahead. She might be OOF traveling or buried in work ⛏️. When she sees this, she'll let us know if she has a patch request.

@guardrex guardrex merged commit 3e99746 into dotnet:main Oct 30, 2025
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.

3 participants