3 ms·
I’ve introduced the `absence` trigger that does that If express is present but helmet is missing then break. Do you think that’d help achieve what you have in
by cfabianski 4y ago
I’ve introduced the `absence` trigger that does that
If express is present but helmet is missing then break.
Do you think that’d help achieve what you have in mind?
- spmurrayzzz 4y agoI think the design flaw in most of the problematic rules was from too simple of regex matching. Looking for a string pattern should be a clue to do some deeper analysis (maybe verify via AST), not necessarily to flag the string alone as security failure.
- didroe 4y agoThe rules do work on the AST but the current cookie rule is not as advanced as it could/should be. For example, we really should treat encryption as sanitizing the value. We'll take another look at the rules with this in mind. If you are able to share the (rough) approach you take to build the cookie string it would help us to ensure we're covering the specific case(s) you have.