Code review comment for ~allenpthuang/ubuntu-cve-tracker:usn-oval-generator

Revision history for this message
Allen Huang (allenpthuang) wrote :

> This change seems to be affecting the <cve> tag, specially in the usns field.
> Could you check it please?

Hi Eduardo,

Just want to confirm, do you mean `USN.cves`? The attribute was set to a list of strings (of CVEs). In my change, it's set to a dict, with keys as CVE identifiers and values as CVE objects. Do you think we should keep a separate attribute for CVE objects? Let me know if I misunderstand your comment, thanks!

« Back to merge proposal