Skip to content
New issue

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

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

Already on GitHub? # to your account

Cleanse fileId for logging #105

Merged
merged 2 commits into from
Dec 18, 2024
Merged

Cleanse fileId for logging #105

merged 2 commits into from
Dec 18, 2024

Conversation

rpoet-jh
Copy link
Contributor

Fixes issue of logging user controlled text.

@rpoet-jh rpoet-jh self-assigned this Dec 18, 2024
@rpoet-jh rpoet-jh requested a review from markpatton December 18, 2024 20:40
Copy link
Contributor

@markpatton markpatton left a comment

Choose a reason for hiding this comment

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

Looks sensible.

@rpoet-jh rpoet-jh linked an issue Dec 18, 2024 that may be closed by this pull request
@rpoet-jh rpoet-jh merged commit d05660f into main Dec 18, 2024
3 checks passed
@rpoet-jh rpoet-jh deleted the rdp-1099-clean-log-text branch January 16, 2025 14:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change code to not log user-controlled data
2 participants