Skip to content
This repository has been archived by the owner on May 14, 2020. It is now read-only.

Consistent support for the "ver" action #650

Closed
dune73 opened this issue Nov 18, 2016 · 7 comments · Fixed by #1750
Closed

Consistent support for the "ver" action #650

dune73 opened this issue Nov 18, 2016 · 7 comments · Fixed by #1750
Assignees

Comments

@dune73
Copy link
Contributor

dune73 commented Nov 18, 2016

Many rules have support for the ver action, but far from all. Namely the 949 rules are lacking.

We should add consistent support.

@fzipi
Copy link
Contributor

fzipi commented Sep 28, 2019

@dune73 Do you want to add the same version for all?

In that case, we may want to use a var and just reference it everywhere.

@airween Should be capable of changing everything in no time using the parser :)

@airween airween self-assigned this Sep 28, 2019
@airween
Copy link
Contributor

airween commented Sep 28, 2019

Sure! :)

I assigned this issue for myself, let's discuss about the details.

@dune73
Copy link
Contributor Author

dune73 commented Sep 29, 2019

Let's discuss it at the meeting.

@annawinkler
Copy link
Contributor

I'd like to help out with this one!

@fgsch
Copy link
Contributor

fgsch commented Nov 4, 2019

Meeting outcome: @airween with make sure this issue is solved together with Anna.

@airween
Copy link
Contributor

airween commented Feb 3, 2020

There is a table about occurrence of ver actions (and with values). The point is that all rule must have a ver action? And what would be the correct value now? OWASP_CRS/3.2.0?

@dune73
Copy link
Contributor Author

dune73 commented Mar 4, 2020

Decision during the CRS project chat on March 2, 2020: @airween will bring some consistency to this.

#1683 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
5 participants