Merge ~pfsmorigo/ubuntu-cve-tracker:pfsmorigo/publish_cves_esm_status into ubuntu-cve-tracker:master

Proposed by Paulo Flabiano Smorigo
Status: Merged
Merged at revision: 02d77159047dd6854f3f5d3064d48cca77a9ce7a
Proposed branch: ~pfsmorigo/ubuntu-cve-tracker:pfsmorigo/publish_cves_esm_status
Merge into: ubuntu-cve-tracker:master
Diff against target: 28 lines (+8/-8)
1 file modified
scripts/publish-cves-to-website-api.py (+8/-8)
Reviewer Review Type Date Requested Status
Alex Murray Approve
Review via email: mp+438551@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alex Murray (alexmurray) wrote :

LGTM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/scripts/publish-cves-to-website-api.py b/scripts/publish-cves-to-website-api.py
2index c8e1450..cc143f9 100755
3--- a/scripts/publish-cves-to-website-api.py
4+++ b/scripts/publish-cves-to-website-api.py
5@@ -115,16 +115,16 @@ def post_single_cve(cve_filename):
6 for codename in cve_lib.releases + ["upstream"]:
7 status = None
8
9- # If release is EOL and there is a public update for it
10- if codename in cve_lib.eol_releases \
11- and codename in cve_data["pkgs"][pkg] \
12- and cve_data["pkgs"][pkg][codename] == "released":
13+ # Set the public release first
14+ if codename in cve_data["pkgs"][pkg]:
15 status = cve_data["pkgs"][pkg][codename]
16
17- else:
18- release = codename
19- for release in [codename + "/esm", "esm-infra/" + codename, "esm-apps/" + codename, "ros-esm/" + codename, codename]:
20- if release in cve_data["pkgs"][pkg]:
21+ # If the release is EOL or there is an ESM update for it use ESM status
22+ if codename in cve_lib.get_active_releases_with_esm():
23+ for release in [codename + "/esm", "esm-infra/" + codename,
24+ "esm-apps/" + codename, "ros-esm/" + codename, codename]:
25+ if release in cve_data["pkgs"][pkg] and \
26+ (codename in cve_lib.eol_releases or cve_data["pkgs"][pkg][release] == "released"):
27 status = cve_data["pkgs"][pkg][release]
28 break
29

Subscribers

People subscribed via source and target branches