Use new error type & structured log in AddPeerID - Githubissues

hyperledger-labs / perun-node

State channel node of the blockchain-agnostic state channels framework Perun.

https://labs.hyperledger.org/perun-doc/

Apache License 2.0

18 stars 12 forks source link

Use new error type & structured log in AddPeerID #148

Closed manoranjith closed 3 years ago

manoranjith commented 3 years ago

Description

For AddPeerID API:

Update it to return the newly defined error type. To identify specific error cases, added error constants in the "ID provider" package.
Log the request, error/success response are logged in a structured format (#133).

Category

Implementation Task

Relevant issue

Relates to #132, #133.

Testing

Steps to run the tests

Updated test Test_YAML_Write_Read in idprovider/local package for the newly added error constants.
Updated test Test_Session_AddPeerID in session package and Test_Integ_New in node package for using new error type in AddPeerID api.

Checklist

[x] Name is added to the NOTICE file, if it is not present already.
[x] Changes are rebased onto the target branch.

manoranjith commented 3 years ago

Rebased onto latest develop.