Skip to content

Reject identity point or small order public keys - #93

Open
neilalexander wants to merge 1 commit into
mainfrom
neil/identitypoint
Open

neilalexander wants to merge 1 commit into
mainfrom
neil/identitypoint

Conversation

@neilalexander

Copy link
Copy Markdown
Member

This is extra key validation. I would prefer to have not had to add an extra dependency for this, but the crypto library doesn't export these operations and I trust Filippo.

Signed-off-by: Neil Twigg neil@nats.io

Signed-off-by: Neil Twigg <neil@nats.io>
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 41821123697

Coverage decreased (-0.2%) to 79.956%

Details

  • Coverage decreased (-0.2%) from the base build.
  • Patch coverage: 2 uncovered changes across 1 file (14 of 16 lines covered, 87.5%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
strkey.go 16 14 87.5%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 454
Covered Lines: 363
Line Coverage: 79.96%
Coverage Strength: 68.79 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants