Closed mtruj013 closed 1 year ago
Merging #131 (c68f1f1) into main (f87753d) will not change coverage. The diff coverage is
n/a
.:exclamation: Current head c68f1f1 differs from pull request most recent head 8056028. Consider uploading reports for the commit 8056028 to get more accurate results
@@ Coverage Diff @@
## main #131 +/- ##
=======================================
Coverage 83.34% 83.34%
=======================================
Files 29 29
Lines 1405 1405
=======================================
Hits 1171 1171
Misses 234 234
Flag | Coverage Δ | |
---|---|---|
python | 83.34% <ø> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
webapp/views.py | 77.36% <ø> (ø) |
Done
Rationale
This is breaking all text queries made against CVEs because for reasons that are still not clear to me CVE.notes["author"] and CVE.notes["note"] are different class types than all the other text based fields. I'm still investigating how to fix it, but for now this will at least allow users to submit queries.
QA
dotrun
. If you get errorsdotrun start-db
and thendotrun
should workhttp://0.0.0.0:8030/security/
Issue / Card
Fixes https://warthogs.atlassian.net/browse/WD-5010