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.

Sign up for GitHub

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

Jump to bottom

Log Cleansing of Authkey in indexer POST and Response #280

Closed
1 task
bakerboy448 opened this issue Jun 25, 2021 · 4 comments
Closed
1 task

Log Cleansing of Authkey in indexer POST and Response #280

bakerboy448 opened this issue Jun 25, 2021 · 4 comments
Labels
Status: Maybe One Day Type: Bug Issue is a bug

Comments

@bakerboy448
Copy link
Contributor

bakerboy448 commented Jun 25, 2021 •
edited by azure-boards bot

  • ToDo

backed out of #215

Figure out why this test case passes when the logs were not cleansed

[TestCase(@" href=""feeds.php?feed=torrents_lossless24&user=mySecret&auth=mySecret&passkey=mySecret&authkey=2b51db35e1910123321025a12b9933d2""")]

image

AB#27
@bakerboy448 bakerboy448 added Type: Bug Issue is a bug Area: Logging Status: Help Wanted Help Wanted, no immediate plans for effort on this labels Jun 25, 2021
@Qstick Qstick added Status: Confirmed Stops stale bot from asking if stale and removed Status: Confirmed Stops stale bot from asking if stale labels Jun 26, 2021
@bakerboy448

This comment has been minimized.

Sign in to view
@bakerboy448 bakerboy448 mentioned this issue Aug 11, 2021
Closed
1 task
@bakerboy448 bakerboy448 changed the title Log Cleansing of Authkey in indexer response Log Cleansing of Authkey in indexer POST and Response Aug 31, 2021
@bakerboy448

This comment has been minimized.

Sign in to view
@bakerboy448 bakerboy448 mentioned this issue Aug 31, 2021
Closed
1 task
@Qstick Qstick mentioned this issue Aug 31, 2021
Closed
1 task
bakerboy448 added a commit that referenced this issue Aug 31, 2021
@bakerboy448
Fixed: Better Log Cleansing
665e1f3 
Ref #280 YGG Comment
@bakerboy448 bakerboy448 mentioned this issue Aug 31, 2021
Merged
1 task
Qstick pushed a commit that referenced this issue Aug 31, 2021
@bakerboy448 @Qstick
Fixed: Better Log Cleansing
043b1a0 
Ref #280 YGG Comment
@bakerboy448
Copy link
Contributor Author

bakerboy448 commented Sep 1, 2021 •
edited

Per taloth likely due to html preventing a match

@bakerboy448
Copy link
Contributor Author

Closing for now; can re-open if we see this again.

@bakerboy448 bakerboy448 added Status: Maybe One Day and removed Status: Help Wanted Help Wanted, no immediate plans for effort on this labels Nov 19, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Status: Maybe One Day Type: Bug Issue is a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Qstick @bakerboy448