Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix(log): remove code from services debug logs #162

Merged
merged 1 commit into from
May 16, 2024

Conversation

palazzem
Copy link
Owner

@palazzem palazzem commented May 16, 2024

Related Issues

  • n/a

Proposed Changes:

Removes code from debug logs, to prevent credentials leak.

Testing:

n/a

Extra Notes (optional):

n/a

Checklist

  • Related issues and proposed changes are filled
  • Tests are defining the correct and expected behavior
  • Code is well-documented via docstrings

@palazzem palazzem added this to the 2.4.0 milestone May 16, 2024
@palazzem palazzem self-assigned this May 16, 2024
@palazzem palazzem force-pushed the palazzem/code-error-log branch from b11d7fc to 72946fd Compare May 16, 2024 11:11
@coveralls
Copy link

Pull Request Test Coverage Report for Build 9111246153

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 93.914%

Totals Coverage Status
Change from base Build 9111238569: 0.0%
Covered Lines: 674
Relevant Lines: 721

💛 - Coveralls

@palazzem palazzem merged commit 2d71539 into main May 16, 2024
5 checks passed
@palazzem palazzem deleted the palazzem/code-error-log branch May 16, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants