labml icon indicating copy to clipboard operation
labml copied to clipboard

Add option to exclude "updated" in brakeman.ignore

Open tobyhs opened this issue 10 months ago • 4 comments

The "updated" entry in brakeman.ignore is prone to merge conflicts, so I'm adding an option that allows users to omit it.

tobyhs avatar Apr 07 '24 00:04 tobyhs

Hi there :wave:, @dryrunsecurity here, below is a summary of our analysis and findings.

DryRun Security Status Findings
Secrets Analyzer (beta) :white_check_mark: 0 findings
Authn/Authz Analyzer :white_check_mark: 0 findings
Configured Codepaths Analyzer :white_check_mark: 0 findings
Sensitive Files Analyzer :white_check_mark: 0 findings

[!Note] :green_circle: Risk threshold not exceeded.

[!Tip] Get answers to your security questions. Add a comment in this PR starting with @dryrunsecurity. For example...

@dryrunsecurity What are common security issues with web application cookies?

Powered by DryRun Security

dryrunsecurity[bot] avatar Apr 07 '24 00:04 dryrunsecurity[bot]

I'm actually tempted to drop the timestamp altogether instead of adding a very specific option.

presidentbeef avatar Jul 14 '24 05:07 presidentbeef

I'm actually tempted to drop the timestamp altogether instead of adding a very specific option.

That seems reasonable. I only implemented an option because removing the timestamp altogether breaks compatibility, but I'm not sure if anyone really relies on the updated entry in brakeman.ignore.

tobyhs avatar Jul 14 '24 21:07 tobyhs

I opened a pull request to drop the updated entry altogether at https://github.com/presidentbeef/brakeman/pull/1860

tobyhs avatar Jul 21 '24 01:07 tobyhs