Skip to content

Commit 4e24e25

Browse files
authored
Merge pull request #158 from marcominerva/develop
Refactor security scheme naming in AuthDocumentTransformer
2 parents cc0ca95 + 2d35869 commit 4e24e25

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/SimpleAuthentication/OpenApi/AuthenticationDocumentTransformer.cs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,18 @@ static void CheckAddApiKey(OpenApiDocument document, IConfigurationSection secti
5353

5454
if (!string.IsNullOrWhiteSpace(settings.HeaderName))
5555
{
56-
AddSecurityScheme(document, $"{settings.SchemeName} in Header", SecuritySchemeType.ApiKey, null, ParameterLocation.Header, settings.HeaderName, "Insert the API Key");
57-
AddSecurityRequirement(document, $"{settings.SchemeName} in Header");
56+
var schemeName = $"{settings.SchemeName}-Header";
57+
58+
AddSecurityScheme(document, schemeName, SecuritySchemeType.ApiKey, null, ParameterLocation.Header, settings.HeaderName, "Insert the API Key");
59+
AddSecurityRequirement(document, schemeName);
5860
}
5961

6062
if (!string.IsNullOrWhiteSpace(settings.QueryStringKey))
6163
{
62-
AddSecurityScheme(document, $"{settings.SchemeName} in Query String", SecuritySchemeType.ApiKey, null, ParameterLocation.Query, settings.QueryStringKey, "Insert the API Key");
63-
AddSecurityRequirement(document, $"{settings.SchemeName} in Query String");
64+
var schemeName = $"{settings.SchemeName}-QueryString";
65+
66+
AddSecurityScheme(document, schemeName, SecuritySchemeType.ApiKey, null, ParameterLocation.Query, settings.QueryStringKey, "Insert the API Key");
67+
AddSecurityRequirement(document, schemeName);
6468
}
6569
}
6670

0 commit comments

Comments
 (0)