Skip to content

Prototype - Support for configuartion of curve to use#20

Draft
mbwhite wants to merge 1 commit into
IBM:mainfrom
mbwhite:idemix-config-update
Draft

Prototype - Support for configuartion of curve to use#20
mbwhite wants to merge 1 commit into
IBM:mainfrom
mbwhite:idemix-config-update

Conversation

@mbwhite
Copy link
Copy Markdown
Contributor

@mbwhite mbwhite commented Oct 24, 2022

Note PR for discussion of the approach (and being new to the codebase!)

  • Needs an update to the idemix config protobuf to add in a CurveId field
  • If a idemixconfig.yaml is present (peer of the existing files read) read, and take the curve specified in there as the one that needs to be used
  • This is read and returned in the MSPConfig struct
  • When the Idemix instance is setup it gets this MSPConfig struct passed so can create idemix as needed
  • Note this meant some init code in new has moved to setup

Backward compatibility, if the file isn't there, or the field isn't there or known about, the original curve value is used.

Mathlib exports an enumeration for curveids, though these are ints - no ammenable to be used in a configuration file, therefore adopted a simple string naming. Ideally these should also be constants within mathlib

Signed-off-by: Matthew B White whitemat@uk.ibm.com

Signed-off-by: Matthew B White <whitemat@uk.ibm.com>
@mbwhite mbwhite force-pushed the idemix-config-update branch from b5c0194 to 51f116c Compare December 13, 2022 16:07
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.

1 participant