davidkel / hlf

0 stars 0 forks source link

[Fabric-Network] Logging should include a transaction id for submit/execute #18

Open davidkel opened 4 years ago

davidkel commented 4 years ago

We need to be able to track a request end to end and the transaction id is the way to do this. We should also look at the lower level node-sdk logging methods as well to include a transaction id.

I don't believe we need to include the full transaction id, probably the first 8 characters are enough but we should be consistent with how core fabric logs things. We should also ensure that as part of the hardening and serviceability work going into 1.4 that we help drive better logging at the correct level in core fabric and chaincode as well.

In fact, from experience I think we need to include not only the txid, but also the channel id and the function being invoked.