Skip to content

Redacts a copy of the parsed GitHub token for logging - #435

Open
nikkoc wants to merge 1 commit into
aws:mainfrom
nikkoc:main
Open

nikkoc wants to merge 1 commit into
aws:mainfrom
nikkoc:main

Conversation

@nikkoc

@nikkoc nikkoc commented Oct 29, 2026

Copy link
Copy Markdown

Fixes issue #434

  • Clones the parsed data object for Github Access Token sanitation in logging so that the actual data does not get redacted.
  • Updates the github_token_redaction unit test to check the actual token is not redacted but that any logging is redacted.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

…re it is used.

* Updates the github_toekn_redaction test to check the actual token is not redacted but that any logging is redacted
@nikkoc nikkoc changed the title Fixes issue #434 by redacting a copy of the parsed data for logging Redacts a copy of the parsed GitHub token for logging Oct 30, 2026
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