Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FAB-17170 Peer CLI should encode mdata lowercase #385

Merged
merged 1 commit into from
Dec 10, 2019
Merged

FAB-17170 Peer CLI should encode mdata lowercase #385

merged 1 commit into from
Dec 10, 2019

Commits on Dec 10, 2019

  1. FAB-17170 Peer CLI should encode mdata lowercase 

    Once upon a time, the JSON fields in the chaincode package metadata.json
were upper case.  The peer CLI was written to reflect this fact.  When
they were changed to lower-case, the peer CLI was not updated, but
because of the fuzzy JSON matching rules in the golang JSON decoded it
was not noticed.  However, for external builders, the inconsistency is
more likely to be noticed as tools like jq etc. may not default to this
same fuzzy case-insensitive matching.

This change simply modifies the peer CLI to encode the metadata fields
as lower case.

Signed-off-by: Jason Yellick <jyellick@us.ibm.com>
    Jason Yellick committed Dec 10, 2019
    Configuration menu
    Copy the full SHA
    84845ff View commit details
    Browse the repository at this point in the history