Skip to content
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

Missing escape in CVS v3 #163

Open
kurtseifried opened this issue Jun 24, 2023 · 2 comments
Open

Missing escape in CVS v3 #163

kurtseifried opened this issue Jun 24, 2023 · 2 comments

Comments

@kurtseifried
Copy link
Contributor

"pattern": "CVSS:3\.[0-9]/AV:[NALP]\/AC:[LH]\/PR:[NLH]\/UI:[NR]\/S:[UC]\/C:[NLH]\/I:[NLH]\/A:[NLH]"

should be "pattern": "CVSS:3\.[0-9]\/AV:[NALP]\/AC:[LH]\/PR:[NLH]\/UI:[NR]\/S:[UC]\/C:[NLH]\/I:[NLH]\/A:[NLH]"

fixed in a PR.

But more worryingly this indicates a lack of test data/harness, is there a test harness/data somewhere outside of this repo?

@kurtseifried
Copy link
Contributor Author

kurtseifried commented Jun 24, 2023

#165

something went bonkers between vs code and github desktop, fixed.

@andrewpollock
Copy link
Contributor

But more worryingly this indicates a lack of test data/harness, is there a test harness/data somewhere outside of this repo?

Not to my knowledge, @oliverchang ?

I think we can call this specific issue resolved by #165

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants