Skip to content

Commit 52417ab

Browse files
committed
added comment on nosec issue
1 parent a3dd40f commit 52417ab

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

apps/internal/oauth/ops/accesstokens/accesstokens.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ func (c *Credential) JWT(ctx context.Context, authParams authority.AuthParams) (
156156
// https://tools.ietf.org/html/rfc7517#section-4.8
157157
func thumbprint(cert *x509.Certificate, alg string) []byte {
158158
switch alg {
159-
case jwt.SigningMethodRS256.Name:
159+
case jwt.SigningMethodRS256.Name: // identity providers like ADFS don't support SHA256 assertions, so need to support this
160160
hash := sha1.Sum(cert.Raw) /* #nosec */
161161
return hash[:]
162162
default:

0 commit comments

Comments
 (0)