Signed-off-by: Giulia Naponiello gnaponie@redhat.com
rebased onto 72ffa4abaef6126eea9bd68738fa15dfa82e406b
rebased onto cc6287dfe37432a4869bc993bd00adbf88be361d
@jkaluza I've removed verify=False. If it's a problem only on my machine it should work.
It's time to let the past go.
How about something like:
list with one element, which is a dict with two keys, "impact", and "affects". The first references the impact of the CVE, and the second is a list of dicts representing packages affected by the CVE.
Also, if we're expecting a one-item list, why not just modify this method to return that one item instead of the list?
Could you use log,exception here instead? In case of error like this, I want to see the full exception. Or do you presume this warning will happen often and is not sign of real error "somewhere"?
log,exception
rebased onto 140a700b00c392e2b7bd75e2bf9b5079979d5e2d
Thank you for the reviews. I've addressed all the comments.
+1.
Commit fd58f21f fixes this pull-request
Pull-Request has been merged by jkaluza
Signed-off-by: Giulia Naponiello gnaponie@redhat.com