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

Endorsing functionality - endorser side #13

Closed
esune opened this issue May 5, 2021 · 3 comments · Fixed by #16 or #11
Closed

Endorsing functionality - endorser side #13

esune opened this issue May 5, 2021 · 3 comments · Fixed by #16 or #11
Assignees

Comments

@esune
Copy link
Member

esune commented May 5, 2021

There currently is no way for an aries-cloudagent started with endorser capabilities to use auto flags to streamline the endorsing process, so implementing a minimal controller is necessary.

The controller will need to expose an endpoint to receive webhooks from the endorser agent, and will need to handle the following scenarios:

  1. new connection established: once a new connection is established, the endorser needs to set some metadata using the admin api endpoint ​/transactions​/{conn_id}​/set-endorser-role. The role that must be set is TRANSACTION_ENDORSER.
  2. once a new transaction requiring endorsing is received, the endorser should just go ahead, endorse it and return it to the author.

Please note that for number 2 it would be wise to design the process so that a manual control could be easily plugged-in if necessary (e.g.: user-controlled gates driven by a web UI)

@amanji
Copy link
Contributor

amanji commented May 12, 2021

@esune Is there any missing functionality here as described? If so, do we want to open a separate ticket to track those things specifically and close this or should we keep this open?

@esune
Copy link
Member Author

esune commented May 12, 2021

@esune Is there any missing functionality here as described? If so, do we want to open a separate ticket to track those things specifically and close this or should we keep this open?

I think we're good for now. The only thing we did not implement is API endpoints for an admin dashboard, but that should be covered by bcgov/dts-endorser-service#26 I think

@esune
Copy link
Member Author

esune commented May 12, 2021

Resolved by #11 when merged

@esune esune linked a pull request May 12, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants