Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Make cas-url arg mandatory in orb-cli vct verify #1308

Merged
merged 1 commit into from
May 26, 2022

Conversation

bstasyszyn
Copy link
Contributor

A cas-url must be specified in order to verify the proof in the VCT log. The CAS REST endoint of Orb works with both local and IPFS CAS.

Also, updated the README.md file in the tutorial to reference Read the Docs so as to avoid duplication.

closes #1305

Signed-off-by: Bob Stasyszyn Bob.Stasyszyn@securekey.com

A cas-url must be specified in order to verify the proof in the VCT log. The CAS REST endoint of Orb works with both local and IPFS CAS.

Also, updated the README.md file in the tutorial to reference Read the Docs so as to avoid duplication.

closes trustbloc#1305

Signed-off-by: Bob Stasyszyn <Bob.Stasyszyn@securekey.com>
@cla-bot cla-bot bot added the cla-signed label May 25, 2022
@codecov
Copy link

codecov bot commented May 25, 2022

Codecov Report

Merging #1308 (9c8f574) into main (8015c1a) will decrease coverage by 0.05%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main    #1308      +/-   ##
==========================================
- Coverage   87.93%   87.87%   -0.06%     
==========================================
  Files         190      190              
  Lines       18302    18294       -8     
==========================================
- Hits        16093    16075      -18     
- Misses       1405     1413       +8     
- Partials      804      806       +2     
Impacted Files Coverage Δ
cmd/orb-cli/vctcmd/vct.go 100.00% <ø> (ø)
cmd/orb-cli/vctcmd/verifycmd.go 74.53% <80.00%> (-2.39%) ⬇️
...pub/service/inbox/httpsubscriber/httpsubscriber.go 88.88% <0.00%> (-8.89%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8015c1a...9c8f574. Read the comment docs.

@bstasyszyn bstasyszyn merged commit f7bea4a into trustbloc:main May 26, 2022
@bstasyszyn bstasyszyn deleted the 1305 branch May 26, 2022 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add IPFS support for orb-cli 'vct verify' command
2 participants