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

Return a SignedResponse from the client query-storage-deal command #3389

Merged

Conversation

ingar
Copy link
Contributor

@ingar ingar commented Sep 11, 2019

Summary

Returned a storagedeal.SignedResponse from query-storage-deal.

Sample Output

{
  "State": 7,
  "Message": "",
  "ProposalCid": {
    "/": "bafy2bzacebwfakwyxdnhwfhtapwkerpmps4up5hkte3pmopyteoqexm6foxz4"
  },
  "ProofInfo": {
    "SectorID": 1,
    "CommD": "dZ2edPVzA1nSK0PpmFfz4DiTanhYjpchghwN8d1oeg8=",
    "CommR": "OodhwUTX1buSWDjZqql0rf2DRJvRmraEbYGyHLakIGg=",
    "CommRStar": "aFZOviu+UJkAuMYGLQMzjNlealquZ7BqKI3tX/2YQF4=",
    "CommitmentMessage": {
      "/": "bafy2bzaceaa4wo5zktetxmm5llsuqhnytsek5wcenhpums6cilovqaady3txg"
    },
    "PieceInclusionProof": "EAAAAAAAAABEral84RHiyMCNHov3CfFrz7fyFnBV7SIim5V6oiSdaw=="
  },
  "Signature": "Fl40oZ5WOpRd4sOeWqy+dnt08jkO+yLWmmnqOse100Z1IihVmWp1vUfEEreV6p1eJ2UF9hysL3A3d3baGMNjLwE="
}

@ingar ingar requested review from acruikshank and removed request for anorth September 11, 2019 17:19
@ingar ingar force-pushed the bugfix/3388-return-signed-response-from-query-storage-deal branch from cf7a099 to 4cc8708 Compare September 11, 2019 17:20
@codecov-io
Copy link

codecov-io commented Sep 11, 2019

Codecov Report

Merging #3389 into master will increase coverage by <1%.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master   #3389    +/-   ##
=======================================
+ Coverage      47%     47%   +<1%     
=======================================
  Files         228     228            
  Lines       14235   14235            
=======================================
+ Hits         6809    6812     +3     
+ Misses       6425    6421     -4     
- Partials     1001    1002     +1

Copy link
Contributor

@acruikshank acruikshank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@ingar ingar merged commit 82937c7 into master Sep 11, 2019
@ingar ingar deleted the bugfix/3388-return-signed-response-from-query-storage-deal branch September 11, 2019 17:42
@hazmatzo
Copy link

Woot

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.

5 participants