-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Feat] Detector implementation for Azure SAS Tokens #3963
base: main
Are you sure you want to change the base?
[Feat] Detector implementation for Azure SAS Tokens #3963
Conversation
used background context for individual test.
` | ||
) | ||
|
||
func TestAzureSASToken_Pattern(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO this needs more positive and negative test cases. SAS tokens can be long and complicated, a single test isn't sufficient.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added more test cases to increase the coverage.
Description:
This PR implements a new Detector for Azure SAS Token. This detector looks for blob URL with signature, generated via portal, and verify the credentials based on generated values. Please note that, The Detector type was already mentioned with the implementation. This PR reuse that type from proto
Test Output:
Checklist:
make test-community
)?make lint
this requires golangci-lint)?