Skip to content

Move HTTP trace logging docs to APPENDIX-I#101

Merged
acn-ericlaw merged 1 commit intomainfrom
feature/fix-http-trace-logging-docs
Apr 20, 2026
Merged

Move HTTP trace logging docs to APPENDIX-I#101
acn-ericlaw merged 1 commit intomainfrom
feature/fix-http-trace-logging-docs

Conversation

@jerryyanmj
Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #99. Relocates the HTTP trace logging documentation to a more appropriate place:

  • Removed from guides/APPENDIX-III.md — the async HTTP request parameter reference is not the right place for logging configuration
  • Removed from guides/CONFIGURATION-REFERENCE.mdHTTP_TRACE_LEVEL is a logger-level env var, not a platform configuration key
  • Added to guides/APPENDIX-I.md — placed immediately after the existing "Distributed trace" section, which already covers the same log4j2.xml pattern for enabling debug-level logging

Changed files

File Change
guides/APPENDIX-I.md Added "HTTP request and response trace logging" section after "Distributed trace"
guides/APPENDIX-III.md Removed misplaced example flows and HTTP trace logging section
guides/CONFIGURATION-REFERENCE.md Removed HTTP_TRACE_LEVEL row from HTTP Client table

🤖 Generated with Claude Code

- Remove HTTP trace logging section from APPENDIX-III (wrong file)
- Remove HTTP_TRACE_LEVEL row from CONFIGURATION-REFERENCE.md (not a config key)
- Add "HTTP request and response trace logging" section to APPENDIX-I,
  placed after the existing "Distributed trace" section where log4j2.xml
  configuration is already discussed

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jerryyanmj jerryyanmj requested a review from acn-ericlaw April 20, 2026 13:40
Copy link
Copy Markdown
Contributor

@acn-ericlaw acn-ericlaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@acn-ericlaw acn-ericlaw merged commit 3e7d0db into main Apr 20, 2026
2 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