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(connector-fabric): identity json signing credentials #1130 #1132

Merged
merged 1 commit into from
Jul 22, 2021

Conversation

petermetz
Copy link
Contributor

Introduces a new, optional (for now) parameter on the run tx
endpoint's request object called gatewayOptions which is capable
of including everything that one needs to instantiate a gateway
object of the underlying Fabric Node SDK.

This change makes it possible to not need the keychain plugin
at all when someone does not need/want it.

Fixes #1130

Depends on #1124

Signed-off-by: Peter Somogyvari peter.somogyvari@accenture.com

@petermetz petermetz self-assigned this Jul 14, 2021
@petermetz petermetz added enhancement New feature or request Fabric and removed dependent labels Jul 14, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #1132 (a4f2bc2) into main (6d1de27) will decrease coverage by 0.05%.
The diff coverage is 72.50%.

❗ Current head a4f2bc2 differs from pull request most recent head da94065. Consider uploading reports for the commit da94065 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1132      +/-   ##
==========================================
- Coverage   73.14%   73.09%   -0.06%     
==========================================
  Files         246      247       +1     
  Lines        8617     8664      +47     
  Branches      996     1008      +12     
==========================================
+ Hits         6303     6333      +30     
- Misses       1783     1790       +7     
- Partials      531      541      +10     
Impacted Files Coverage Δ
...pescript/generated/openapi/typescript-axios/api.ts 86.13% <ø> (ø)
...abric/src/main/typescript/common/create-gateway.ts 64.86% <64.86%> (ø)
.../main/typescript/plugin-ledger-connector-fabric.ts 82.64% <68.96%> (-0.17%) ⬇️
...rc/main/typescript/fabric/fabric-test-ledger-v1.ts 83.59% <100.00%> (+0.17%) ⬆️
...t-tooling/src/main/typescript/common/containers.ts 74.48% <0.00%> (-0.83%) ⬇️

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 6d1de27...da94065. Read the comment docs.

Copy link
Contributor

@jonathan-m-hamilton jonathan-m-hamilton left a comment

Choose a reason for hiding this comment

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

LGTM

@petermetz petermetz removed the request for review from kikoncuo July 19, 2021 19:18
@github-actions
Copy link

🎉 Great news! Looks like all the dependencies have been resolved:

💡 To add or remove a dependency please update this issue/PR description.

Brought to you by Dependent Issues (:robot: ). Happy coding!

…-cacti#1130

Introduces a new, optional (for now) parameter on the run tx
endpoint's request object called gatewayOptions which is capable
of including everything that one needs to instantiate a gateway
object of the underlying Fabric Node SDK.

This change makes it possible to not need the keychain plugin
at all when someone does not need/want it.

Fixes hyperledger-cacti#1130

Depends on hyperledger-cacti#1124

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
@petermetz petermetz merged commit bc262a2 into hyperledger-cacti:main Jul 22, 2021
@petermetz petermetz deleted the feat-1130 branch July 22, 2021 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Fabric
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(connector-fabric): identity json signing credentials
4 participants