Merge pull request #412 from pespantelis/main

chore(parser): omit empty values for security schema fields

Author: amazingshovel

openapi/parser/parse_security.go

@@ -60,6 +60,11 @@ func (p *parser) parseSecurityRequirements(requirements ogen.SecurityRequirement
 				return nil, errors.Wrapf(err, "resolve %q", requirementName)
 			}
 
+			var flows ogen.OAuthFlows
+			if spec.Flows != nil {
+				flows = *spec.Flows
+			}
+
 			result = append(result, openapi.SecurityRequirements{
 				Scopes: scopes,
 				Name:   requirementName,
@@ -70,7 +75,7 @@ func (p *parser) parseSecurityRequirements(requirements ogen.SecurityRequirement
 					In:               spec.In,
 					Scheme:           spec.Scheme,
 					BearerFormat:     spec.BearerFormat,
-					Flows:            cloneOAuthFlows(spec.Flows),
+					Flows:            cloneOAuthFlows(flows),
 					OpenIDConnectURL: spec.OpenIDConnectURL,
 				},
 			})

security_spec.go

@@ -18,10 +18,10 @@ type SecuritySchema struct {
 	// by an authorization server, so this information is primarily for documentation purposes.
 	BearerFormat string `json:"bearerFormat,omitempty"`
 	// An object containing configuration information for the flow types supported.
-	Flows OAuthFlows `json:"flows"`
+	Flows *OAuthFlows `json:"flows,omitempty"`
 	// OpenId Connect URL to discover OAuth2 configuration values.
 	// This MUST be in the form of a URL. The OpenID Connect standard requires the use of TLS.
-	OpenIDConnectURL string `json:"openIdConnectUrl"`
+	OpenIDConnectURL string `json:"openIdConnectUrl,omitempty"`
 }
 
 // OAuthFlows allows configuration of the supported OAuth Flows.