Bug #2686
closedFancy Quotes in Documentation
Description
It seems that standard (i.e. ASCII char 22) quotation marks were replaced by fancy quotes in the Payload Keywords, and possibly other, documentation.
https://suricata.readthedocs.io/en/latest/rules/payload-keywords.html#content
With some sections using completely different sets...
Fancy-quotes matched appropriately:
content:“a|0D|bc”;
Fancy-quote mixed with standard quote:
content:”|61 0D 62 63|";
Mismatched fancy quotes:
content:”a|0D|b|63|”;
Is that intentional (i.e. intended to show an indifference for quote types)? If so, that should probably be explicit stated instead.
Even if there is an indifference for quote types, the escaping described will be problematic...
“ |22|
...since it will convert a fancy quote into a standard one, which is likely inappropriate.
Updated by Victor Julien almost 6 years ago
- Status changed from New to Assigned
- Assignee set to Travis Green
- Target version set to 5.0beta1
Updated by Victor Julien over 5 years ago
- Target version changed from 5.0beta1 to 5.0rc1
Updated by Victor Julien over 5 years ago
- Assignee changed from Travis Green to OISF Dev
Updated by Andreas Herz over 5 years ago
https://github.com/OISF/suricata/pull/3901 I also converted the single ones as well.
Is there a good way to prevent this from future PRs?
Updated by Kenneth Kolano over 5 years ago
- Avoid opening the rst files on editors that set fancy quotes
- Avoid copy and paste from sources with fancy quotes
- See if a check for fancy quotes can be added to in the continuous-integration checks
Updated by Andreas Herz over 5 years ago
Kenneth Kolano wrote:
- Avoid opening the rst files on editors that set fancy quotes
- Avoid copy and paste from sources with fancy quotes
This we can just suggest :)
Kenneth Kolano wrote:
- See if a check for fancy quotes can be added to in the continuous-integration checks
This is what I would like to have
Updated by Victor Julien over 5 years ago
- Status changed from Assigned to Closed
- Assignee changed from OISF Dev to Andreas Herz