Skip to content

Conversation

@bestbeforetoday
Copy link
Member

This change changes the Idemix MSP folder structure to match the structure expected by Fabric. It replicates the changes previously delivered (and subsequently reverted) in the following commits:

@bestbeforetoday bestbeforetoday marked this pull request as ready for review November 26, 2025 14:54
@bestbeforetoday bestbeforetoday requested review from a team as code owners November 26, 2025 14:54
@bestbeforetoday
Copy link
Member Author

@adecaro Please could you take a look at this change to confirm I successfully pulled in all the changes you previously made?

@bestbeforetoday
Copy link
Member Author

Feedback from this comment observes that the current change saves the SignerConfig at the root of the organization dir (for example, fabric-samples/test-network/organizations/peerOrganizations/org3.example.com/) instead of the user specific folder (for example, fabric-samples/test-network/organizations/peerOrganizations/org3.example.com/users/Admin@org3.example.com/). This breaks existing peer configurations.

This change changes the Idemix MSP folder structure to match the
structure expected by Fabric. It replicates the changes previously
delivered (and subsequently reverted) in the following commits:

- acea746
- 29e083e
- 0fcf897
- dfd29fe
- 77c54da

Signed-off-by: Mark S. Lewis <Mark.S.Lewis@outlook.com>
Co-authored-by: Angelo De Caro <adc@zurich.ibm.com>
@bestbeforetoday bestbeforetoday force-pushed the idemix-folder-structure branch from a8a77b1 to db7f3bd Compare February 5, 2026 14:50
Also some minor code tidy-up.

Signed-off-by: Mark S. Lewis <Mark.S.Lewis@outlook.com>
@bestbeforetoday bestbeforetoday force-pushed the idemix-folder-structure branch from db7f3bd to 048e6f9 Compare February 5, 2026 16:16
@bestbeforetoday
Copy link
Member Author

@johannww I reverted the change to the location where the SignerConfig is saved. Does this change solve problems for you now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants