Merge pull request #1170 from sashamelentyev/handle-cookie-err

chore: handle cookie error
ogen-go · Jan 31, 2024 · d5fceb0 · d5fceb0
2 parents 092ca33 + 58dd6b1
commit d5fceb0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/gen/_template/security.tmpl b/gen/_template/security.tmpl
@@ -62,13 +62,13 @@ func (s *Server) security{{ $s.Type.Name }}(ctx context.Context, operationName s
 		value := q.Get(parameterName)
         {{- else if $s.Kind.IsCookie }}
 	    var value string
-		switch cookie, err := req.Cookie(parameterName); err {
-		case nil:
+		switch cookie, err := req.Cookie(parameterName); {
+		case err == nil: // if NO error
 			value = cookie.Value
-		case http.ErrNoCookie:
+		case errors.Is(err, http.ErrNoCookie):
 			return ctx, false, nil
 		default:
-			return nil, false, err
+			return nil, false, errors.Wrap(err, "get cookie value")
 		}
 		{{- else }}
             {{ errorf "unexpected security %q:%q" $s.Kind $s.Format }}

diff --git a/internal/integration/test_security/oas_security_gen.go b/internal/integration/test_security/oas_security_gen.go