Skip to content

Sanitize user input with string.Empty#283

Closed
akustn wants to merge 1 commit intoequinor:mainfrom
akustn:sanitize-log-from-user-input
Closed

Sanitize user input with string.Empty#283
akustn wants to merge 1 commit intoequinor:mainfrom
akustn:sanitize-log-from-user-input

Conversation

@akustn
Copy link
Copy Markdown
Contributor

@akustn akustn commented Apr 10, 2026 •
edited
Loading

Closes #275

Ready for review checklist:

  • A self-review has been performed
  • All commits run individually
  • Temporary changes have been removed, like logging, TODO, etc.
  • The PR has been tested locally
  • A test has been written
    • This change doesn't need a new test
  • Relevant issues are linked
  • Remaining work is documented in issues
    • There is no remaining work from this PR that requires new issues
  • The changes do not introduce dead code as unused imports, functions etc.

@akustn akustn self-assigned this Apr 10, 2026
@akustn akustn added improvement Improvement to existing functionality security Issue concerning a security vulnerability labels Apr 10, 2026
@akustn akustn closed this Apr 10, 2026
@akustn akustn deleted the sanitize-log-from-user-input branch April 10, 2026 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@akustn akustn

Labels

improvement Improvement to existing functionality security Issue concerning a security vulnerability

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Sanitize log from user input

1 participant

@akustn