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.

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

Add config debug #444

Merged
merged 1 commit into from
Feb 7, 2025
Merged

Conversation

denyeart
Copy link
Contributor

@denyeart denyeart commented Feb 7, 2025

Changing V1_3 compatibility mode in #435 has caused some user confusion, some additional debug logging will help:

  • Info log FABRIC_CA_SERVER_COMPATIBILITY_MODE_V1_3 setting
  • Debug log when falling back to COMPATIBILITY_MODE_V1_3 authentication header validation

Also take this opportunity to log the entire config upon startup to be consistent with core Fabric:

  • Info log CA config upon startup

@denyeart denyeart requested a review from a team as a code owner February 7, 2025 14:30
bestbeforetoday
bestbeforetoday previously approved these changes Feb 7, 2025
@bestbeforetoday bestbeforetoday enabled auto-merge (squash) February 7, 2025 14:42
@denyeart denyeart force-pushed the config_debug branch 2 times, most recently from 1eb37eb to 13491f0 Compare February 7, 2025 14:47
bestbeforetoday
bestbeforetoday previously approved these changes Feb 7, 2025
- Info log CA config upon startup (consistent with fabric peer approach)
- Info log FABRIC_CA_SERVER_COMPATIBILITY_MODE_V1_3 setting
- Debug log when falling back to COMPATIBILITY_MODE_V1_3 authentication header validation

Signed-off-by: David Enyeart <enyeart@us.ibm.com>
@bestbeforetoday bestbeforetoday merged commit c89dbe8 into hyperledger:main Feb 7, 2025
6 checks passed
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