bevel icon indicating copy to clipboard operation
bevel copied to clipboard

feat(fabric): Upgrade Fabric restserver to latest stable

Open sownak opened this issue 3 years ago • 7 comments

Description

As a developer I want to update the Fabric restserver code in examples/supplychain-app/fabric/chaincode_rest_server/rest-server so that I can remove the security alerts.

Use latest Fabric client and Node js version. Ensure the code is updated and compiles.

Acceptance Criteria

  1. Code compiles successfully.
  2. restserver docker image is created successfully.
  3. Supplychain App works with Fabric network

sownak avatar Feb 18 '22 12:02 sownak

Hi Sownak, I would like to take this issue, If it's okay I would do the necessary changes and upgrade as per the Acceptance criteria. Is there anything specific you are looking for specially in the changes, do let me know

MrRancy avatar May 17 '22 07:05 MrRancy

@MrRancy That would be great! Thanks for your interest. The complete rest-server needs some refresh as per latest Fabric SDK and nodejs. You can start with that and then submit a draft PR. Our team will review and provide comments. Assigning this to you.

sownak avatar May 17 '22 08:05 sownak

is it Yarn or NPM?

MrRancy avatar May 18 '22 11:05 MrRancy

Hi @MrRancy, its NPM

suvajit-sarkar avatar May 23 '22 12:05 suvajit-sarkar

HI @MrRancy , any update on this issue?

suvajit-sarkar avatar Jun 20 '22 12:06 suvajit-sarkar

Hi @suvajit-sarkar, Sorry I was caught up in other priority work the last few weeks in my organization. I will try to finish it off by this week. Apologies. Also, can you elaborate on the Acceptance Criteria, like what are the main things that you are expecting? For example, upgrading all packages to the latest & stable version. Are you using any Static Analysis to scan for security alerts or vulnerabilities?

MrRancy avatar Jun 21 '22 16:06 MrRancy

Hi @sownak @suvajit-sarkar, I am not doing good interms of my health. So unable to push myself in the contribution. Its been so long already, I wanted to let you know guys, I will be opting out myself from the contribution. Sorry for any inconviniences. Also unassigning myself from this issue.

MrRancy avatar Jul 29 '22 20:07 MrRancy

Closing this issue, the rest-server packages were updated to remove the security alerts.

suvajit-sarkar avatar May 22 '23 08:05 suvajit-sarkar