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
Add risk:vuln:id property (SYN-5667) #3447
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3447 +/- ##
==========================================
- Coverage 97.58% 97.46% -0.12%
==========================================
Files 231 231
Lines 47549 47549
==========================================
- Hits 46401 46344 -57
- Misses 1148 1205 +57
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
synapse/models/risk.py
Outdated
@@ -339,6 +339,9 @@ def getModelDefs(self): | |||
('timeline:exploited', ('time', {"ismin": True}), { | |||
'doc': 'The earliest known time when the vulnerability was exploited in the wild.'}), | |||
|
|||
('ext:id', ('str', {'strip': True}), { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the whole vuln is being identified via this ID when it comes in from a 3rd party ( and we already have :reporter
et al, lets just make this :id
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
little tweak
0b39a10
No description provided.