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

feat: Verify that we have latest did information before accepting did… #809

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

sandrask
Copy link
Contributor

… operations

Verify against anchor origin that we have latest did information before accepting did operations (update, recover, deactivate)

Closes #503

Signed-off-by: Sandra Vrtikapa sandra.vrtikapa@securekey.com

… operations

Verify against anchor origin that we have latest did information before accepting did operations (update, recover, deactivate)

Closes trustbloc#503

Signed-off-by: Sandra Vrtikapa <sandra.vrtikapa@securekey.com>
@cla-bot cla-bot bot added the cla-signed label Oct 13, 2021
@codecov
Copy link

codecov bot commented Oct 13, 2021

Codecov Report

Merging #809 (4527a81) into main (789f5f0) will increase coverage by 0.00%.
The diff coverage is 88.05%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #809   +/-   ##
=======================================
  Coverage   89.51%   89.51%           
=======================================
  Files         141      142    +1     
  Lines       12338    12419   +81     
=======================================
+ Hits        11044    11117   +73     
- Misses        799      805    +6     
- Partials      495      497    +2     
Impacted Files Coverage Δ
cmd/orb-server/startcmd/start.go 73.19% <38.46%> (-0.90%) ⬇️
cmd/orb-server/startcmd/params.go 80.54% <81.81%> (+0.03%) ⬆️
pkg/document/util/util.go 90.16% <86.36%> (-9.84%) ⬇️
pkg/document/resolvehandler/resolvehandler.go 95.95% <100.00%> (+2.71%) ⬆️
pkg/document/updatehandler/decorator/decorator.go 100.00% <100.00%> (ø)

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 789f5f0...4527a81. Read the comment docs.

@sandrask sandrask changed the title feat: Verify that we have latest did information before accepting did… chore: Verify that we have latest did information before accepting did… Oct 13, 2021
@sandrask sandrask changed the title chore: Verify that we have latest did information before accepting did… feat: Verify that we have latest did information before accepting did… Oct 13, 2021
@sandrask sandrask merged commit 766d7bd into trustbloc:main Oct 13, 2021
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.

Verify that we have latest did information before accepting did operations
3 participants