-
Notifications
You must be signed in to change notification settings - Fork 299
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
Bug fixes for security agent #843
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #843 +/- ##
===========================================
- Coverage 80.26% 79.73% -0.54%
===========================================
Files 139 134 -5
Lines 12608 12494 -114
===========================================
- Hits 10120 9962 -158
- Misses 2197 2229 +32
- Partials 291 303 +12 ☔ View full report in Codecov by Sentry. |
@@ -163,9 +163,9 @@ func ConfigSecurityValidatorServiceEndPointUrl(url string) ConfigOption { | |||
} | |||
|
|||
// ConfigSecurityDetectionDisableRxss is used to enable or disable RXSS validation. | |||
func ConfigSecurityDetectionDisableRxss(isEnabled bool) ConfigOption { | |||
func ConfigSecurityDetectionDisableRxss(isDisable bool) ConfigOption { |
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.
Recommend renaming isDisable
to isDisabled
to be grammatically correct.
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.
We can change this as we prep for release.
Bug Fixes