multiple set-cookies response header support

Author: randycylonix

pkg/codegen/templates/authenticator.tmpl

@@ -1,4 +1,4 @@
-{{- if gt (len .SecuritySchemeProviderNames) 0 }}
+{{if gt (len .SecuritySchemeProviderNames) 0}}
 type Authenticator interface {
 {{range $ProviderName := .SecuritySchemeProviderNames}}
     {{- $ProviderName | sanitizeGoIdentity | ucFirst}}Auth(r *http.Request) (interface{}, error)

pkg/codegen/templates/constants.tmpl

@@ -1,4 +1,4 @@
-{{- if gt (len .SecuritySchemeProviderNames) 0 }}
+{{if gt (len .SecuritySchemeProviderNames) 0}}
 const (
 {{range $ProviderName := .SecuritySchemeProviderNames}}
     {{- $ProviderName | sanitizeGoIdentity | ucFirst}}Scopes = "{{$ProviderName}}.Scopes"

pkg/codegen/templates/strict/strict-http.tmpl

@@ -35,24 +35,24 @@ const SecurityAuthContextKey = StrictHTTPContextKeyType("Secutiry-Auth")
     // {{$opid}} operation middleware
     func (sh *strictHandler) {{.OperationId}}(w http.ResponseWriter, r *http.Request{{genParamArgs .PathParams}}{{if .RequiresParamObject}}, params {{.OperationId}}Params{{end}}) {
         {{/* __BEGIN_CYLONIX_MOD__ */}}
-        {{- if gt (len .SecurityDefinitions) 0 }}
+        {{- if gt (len .SecurityDefinitions) 0 -}}
             var auth interface{}
-            {{range $i, $s := .SecurityDefinitions}}
-                {{- if gt $i 0 }}
+            {{- range $i, $s := .SecurityDefinitions}}
+                {{- if gt $i 0}}
                     if auth == nil {
-                {{end}}
+                {{- end}}
                 auth, _ = sh.ssi.GetAuthenticator().{{.ProviderName | sanitizeGoIdentity | ucFirst}}Auth(r)
-                {{- if gt $i 0 }}
+                {{- if gt $i 0}}
                     }
-                {{end}}
-            {{end}}
+                {{- end}}
+            {{- end}}
             if auth == nil {
                 http.Error(w, "Unauthorized!", http.StatusUnauthorized)
                 return
             }
             ctx := context.WithValue(r.Context(), SecurityAuthContextKey, auth)
             r = r.WithContext(ctx)
-            {{end}}
+        {{- end}}
         {{/* __END_CYLONIX_MOD__ */}}
 
         var request {{$opid | ucFirst}}RequestObject

pkg/codegen/templates/strict/strict-interface.tmpl

@@ -83,7 +83,16 @@
                     }
                 {{end -}}
                 {{range $headers -}}
-                    w.Header().Set("{{.Name}}", fmt.Sprint(response.Headers.{{.GoName}}))
+                    {{/* __BEGIN_CYLONIX_MOD__*/}}
+                    {{- if and (eq .Name "Set-Cookie") (ne .Schema.ArrayType nil)}}
+                        // Special handling for https://github.com/OAI/OpenAPI-Specification/issues/1237
+                        for _, v := range response.Headers.{{.GoName}} {
+                            w.Header().Add("{{.Name}}", v)
+                        }
+                    {{else -}}
+                        w.Header().Set("{{.Name}}", fmt.Sprint(response.Headers.{{.GoName}}))
+                    {{- end -}}
+                    {{/* __END_CYLONIX_MOD__*/}}
                 {{end -}}
                 w.WriteHeader({{if $fixedStatusCode}}{{$statusCode}}{{else}}response.StatusCode{{end}})
                 {{$hasBodyVar := or ($hasHeaders) (not $fixedStatusCode) (not .IsSupported)}}
@@ -128,7 +137,16 @@
             {{end -}}
             func (response {{$opid}}{{$statusCode}}Response) Visit{{$opid}}Response(w http.ResponseWriter) error {
                 {{range $headers -}}
-                    w.Header().Set("{{.Name}}", fmt.Sprint(response.Headers.{{.GoName}}))
+                    {{/* __BEGIN_CYLONIX_MOD__*/}}
+                    {{- if and (eq .Name "Set-Cookie") (ne .Schema.ArrayType nil)}}
+                        // Special handling for https://github.com/OAI/OpenAPI-Specification/issues/1237
+                        for _, v := range response.Headers.{{.GoName}} {
+                            w.Header().Add("{{.Name}}", v)
+                        }
+                    {{else -}}
+                        w.Header().Set("{{.Name}}", fmt.Sprint(response.Headers.{{.GoName}}))
+                    {{- end -}}
+                    {{/* __END_CYLONIX_MOD__*/}}
                 {{end -}}
                 w.WriteHeader({{if $fixedStatusCode}}{{$statusCode}}{{else}}response.StatusCode{{end}})
                 return nil

pkg/codegen/templates/strict/strict-server.tmpl

@@ -2,23 +2,23 @@
 
 // StrictServer implements StrictServerInterface with handlers to be customized.
 type StrictServer struct {
-Authenticator Authenticator
-{{range .}}{{.SummaryAsComment }}
-// ({{.Method}} {{.Path}})
-{{$opid := .OperationId -}}
-{{$opid}}Handler func(ctx context.Context, request {{$opid | ucFirst}}RequestObject) ({{$opid | ucFirst}}ResponseObject, error)
-{{end}}{{/* range . */ -}}
+    Authenticator Authenticator
+    {{range .}}{{.SummaryAsComment }}
+        // ({{.Method}} {{.Path}})
+        {{$opid := .OperationId}}
+        {{$opid}}Handler func(ctx context.Context, request {{$opid | ucFirst}}RequestObject) ({{$opid | ucFirst}}ResponseObject, error)
+    {{end}}{{/* range . */}}
 }
 
-{{range .}}{{.SummaryAsComment }}
-// ({{.Method}} {{.Path}})
-{{$opid := .OperationId -}}
-func (s *StrictServer) {{$opid}}(ctx context.Context, request {{$opid | ucFirst}}RequestObject) ({{$opid | ucFirst}}ResponseObject, error) {
-	if s.{{$opid}}Handler != nil {
-        return s.{{$opid}}Handler(ctx, request)
+{{range .}}{{.SummaryAsComment}}
+    // ({{.Method}} {{.Path}})
+    {{$opid := .OperationId}}
+    func (s *StrictServer) {{$opid}}(ctx context.Context, request {{$opid | ucFirst}}RequestObject) ({{$opid | ucFirst}}ResponseObject, error) {
+	    if s.{{$opid}}Handler != nil {
+            return s.{{$opid}}Handler(ctx, request)
+        }
+        return nil, errors.New("api {{$opid}} is not implemented")
     }
-    return nil, errors.New("api {{$opid}} is not implemented")
-}
 {{end}}
 
 func (s StrictServer) GetAuthenticator() Authenticator {