Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

perf: remove extra parsing #3566

Merged
merged 3 commits into from
Apr 25, 2023

Conversation

skynet2
Copy link
Contributor

@skynet2 skynet2 commented Apr 25, 2023

verifySignature does not require parsed claims to do that functionality. we already parsing claims in many other places.
image

@codecov
Copy link

codecov bot commented Apr 25, 2023

Codecov Report

Merging #3566 (6d51a58) into main (dbfff0c) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 6d51a58 differs from pull request most recent head 33fe70b. Consider uploading reports for the commit 33fe70b to get more accurate results

@@            Coverage Diff             @@
##             main    #3566      +/-   ##
==========================================
- Coverage   87.76%   87.76%   -0.01%     
==========================================
  Files         351      351              
  Lines       48636    48627       -9     
==========================================
- Hits        42685    42677       -8     
+ Misses       4403     4402       -1     
  Partials     1548     1548              
Impacted Files Coverage Δ
pkg/doc/jwt/verifier.go 76.78% <100.00%> (-1.19%) ⬇️
pkg/doc/signature/verifier/verifier.go 95.00% <100.00%> (ø)
pkg/doc/verifiable/embedded_proof.go 82.52% <100.00%> (-0.50%) ⬇️
pkg/doc/verifiable/linked_data_proof.go 85.93% <100.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@skynet2 skynet2 force-pushed the perf-remove-extraparse branch from f55f41d to a4bbc12 Compare April 25, 2023 13:20
@skynet2 skynet2 force-pushed the perf-remove-extraparse branch 2 times, most recently from 6d51a58 to 33fe70b Compare April 25, 2023 17:46
@fqutishat fqutishat merged commit a8b546e into hyperledger-archives:main Apr 25, 2023
Moopli pushed a commit to Moopli/aries-framework-go that referenced this pull request Apr 25, 2023
* perf: remove extra parsing

* fix: lint

* perf: avoid one more perse
Moopli pushed a commit to Moopli/aries-framework-go that referenced this pull request Apr 25, 2023
* perf: remove extra parsing

* fix: lint

* perf: avoid one more perse
sudeshrshetty pushed a commit that referenced this pull request Apr 25, 2023
* perf: remove extra parsing

* fix: lint

* perf: avoid one more perse

Co-authored-by: Stas Dmytryshyn <iqpirat@gmail.com>
Moopli added a commit to Moopli/aries-framework-go that referenced this pull request Apr 25, 2023
sudeshrshetty added a commit that referenced this pull request Apr 25, 2023
sudeshrshetty pushed a commit that referenced this pull request Apr 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants