Skip to content

hide the details of an exception - #187

Open
cedoucette wants to merge 1 commit into
codeforboston:masterfrom
cedoucette:issue-161
Open

cedoucette wants to merge 1 commit into
codeforboston:masterfrom
cedoucette:issue-161

Conversation

@cedoucette

Copy link
Copy Markdown
Collaborator

to avoid leaking any potential secrets

@cedoucette

Copy link
Copy Markdown
Collaborator Author

fyi - I have not a) reproduced the issue before this change, and b) reproduced the issue after the change

@patrickZWY

Copy link
Copy Markdown
Collaborator

fyi - I have not a) reproduced the issue before this change, and b) reproduced the issue after the change

nah, not really needed to reproduce (I made it sound harder so this deters auto-pr bots). thanks for picking up the issue.

@patrickZWY patrickZWY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@patrickZWY patrickZWY linked an issue Sep 23, 2026 that may be closed by this pull request

This branch has not been deployed

No deployments
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.

Logging may leak slack webhook secret

2 participants