fix(policies): ignore http 405 as well as 404 (#1672)

jiparis · web-flow · commit b15c156bea41 · 2024-12-17T23:57:23.000+01:00
Signed-off-by: Jose I. Paris &lt;jiparis@chainloop.dev&gt;
diff --git a/app/controlplane/pkg/policies/policyprovider.go b/app/controlplane/pkg/policies/policyprovider.go
@@ -138,7 +138,7 @@ func (p *PolicyProvider) ValidateAttachment(att *schemaapi.PolicyAttachment, tok
 	}
 
 	if resp.StatusCode != http.StatusOK {
-		if resp.StatusCode == http.StatusNotFound {
+		if resp.StatusCode == http.StatusNotFound || resp.StatusCode == http.StatusMethodNotAllowed {
 			// Ignore endpoint not found as it might not be implemented by the provider
 			return nil
 		}

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ func (p PolicyProvider) ValidateAttachment(att schemaapi.PolicyAttachment, tok`
`138`	`138`	`}`
`139`	`139`
`140`	`140`	`if resp.StatusCode != http.StatusOK {`
`141`		`- if resp.StatusCode == http.StatusNotFound {`
	`141`	`+ if resp.StatusCode == http.StatusNotFound \|\| resp.StatusCode == http.StatusMethodNotAllowed {`
`142`	`142`	`// Ignore endpoint not found as it might not be implemented by the provider`
`143`	`143`	`return nil`
`144`	`144`	`}`