Skip to content

Commit

Permalink
correct the public key comparison logic
Browse files Browse the repository at this point in the history
  • Loading branch information
ahrtr committed Nov 24, 2021
1 parent e2273f9 commit 63ff6d4
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions server/auth/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ func (opts *jwtOptions) rsaKey() (interface{}, error) {
}

// both keys provided, make sure they match
if pub != nil && pub.E != priv.E && pub.N.Cmp(priv.N) != 0 {
if pub != nil && !pub.Equal(priv.Public()) {
return nil, ErrKeyMismatch
}

Expand Down Expand Up @@ -183,8 +183,7 @@ func (opts *jwtOptions) ecKey() (interface{}, error) {
}

// both keys provided, make sure they match
if pub != nil && pub.Curve != priv.Curve &&
pub.X.Cmp(priv.X) != 0 && pub.Y.Cmp(priv.Y) != 0 {
if pub != nil && !pub.Equal(priv.Public()) {
return nil, ErrKeyMismatch
}

Expand Down

0 comments on commit 63ff6d4

Please sign in to comment.