Merge ~ebarretto/ubuntu-cve-tracker:new-pkg-cache into ubuntu-cve-tracker:master
Proposed by
Eduardo Barretto
Status: | Merged |
---|---|
Merged at revision: | 63db2ef1e8781e5772ed0a22c05db0536a2746ac |
Proposed branch: | ~ebarretto/ubuntu-cve-tracker:new-pkg-cache |
Merge into: | ubuntu-cve-tracker:master |
Diff against target: |
32 lines (+10/-2) 1 file modified
scripts/fetch-db (+10/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
David Fernandez Gonzalez | Approve | ||
Ubuntu Security Team | Pending | ||
Review via email: mp+444614@code.launchpad.net |
Description of the change
This PR alters `scripts/fetch-db` to add a new argument "dbtype" to allow us to fetch package cache from another server.
The way I added the new argument shouldn't affect current services using `fetch-db`.
To post a comment you must log in.
I would prefer the second approach so we only have one script.
We could make an optional flag for this so if this flag appears (something like ---pkg-cache), use https:/ /security- metadata. canonical. com/pkg- cache/"$i".
This way, if we need a new endpoint in the future we can simply add more supported flags. As we are listing this in services/crons/etc, I feel like having to provide the whole URL is messier.