Skip to content

Comments

Fix bug in the extension method 'InferSecuritySchemes' so it will actually enable it#2540

Closed
martijn1279 wants to merge 1 commit intodomaindrivendev:masterfrom
martijn1279:bugfix/InferSecuritySchemes_actually_not_enabled
Closed

Fix bug in the extension method 'InferSecuritySchemes' so it will actually enable it#2540
martijn1279 wants to merge 1 commit intodomaindrivendev:masterfrom
martijn1279:bugfix/InferSecuritySchemes_actually_not_enabled

Conversation

@martijn1279
Copy link

When using the extension method InferSecuritySchemes it wont actually do anything because its not being deepcopied to the new object

options.InferSecuritySchemes(schemes => new Dictionary<string, OpenApiSecurityScheme>
{
    ["OAuth2"] = new OpenApiSecurityScheme
    {
        Type = SecuritySchemeType.OAuth2,
        Name = "OAuth2",
    },
});

Havunen added a commit to Havunen/DotSwashbuckle2 that referenced this pull request Feb 11, 2024
Havunen added a commit to Havunen/DotSwashbuckle that referenced this pull request Feb 18, 2024
@martincostello
Copy link
Collaborator

Thanks for contributing - if you'd like to continue with this pull request, please rebase against the default branch to pick up our new CI. We'd also like a test.

@martincostello
Copy link
Collaborator

Superseded by #2796.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants