Skip to content
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

differentiate security updates #148

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

tpo
Copy link
Contributor

@tpo tpo commented Mar 8, 2018

This patch makes the UI display security updates with a 'U' instead of a 'u'.

The patch is the counterpart to this apt-dater-host commit ...

sourcepole/apt-dater-host@8ac83e0

... which make apt-dater-host report security updates with a flag 'U' instead of a 'u'.

Same as with 8ac83e0, this patch should be understood as a proof of concept that works.

What is missing from this patch is:

a bump of the protocol version
updates to the protocol documentation
updates to report generation
updates to the on screen help

I'd very much appreciate the inclusion of this /feature/. It does not need to be implemented the way I did it. In particular, maybe using a 'U' for signaling a security update could be debated and also the "OR" logic which is used for the signaling of the HOST_STATUS_PKGSECUPDATE flag.

I'd ask to please comment on this patch. (Merging it is of course also very fine ;-)

This implements the feature requested here: #24

…f a 'u'.

This patch is the counterpart to this apt-dater-host commit ...

  sourcepole/apt-dater-host@8ac83e0

... which make apt-dater-host report security updates with
 a flag 'U' instead of a 'u'.
@tpo tpo mentioned this pull request Mar 8, 2018
Patch retrieved from Debian package of apt-dater. Original source:
   http://http.debian.net/debian/pool/main/a/apt-dater/apt-dater_1.0.3-6.debian.tar.xz
Patch created by Patrick Matthäi <[email protected]>
@liske liske self-assigned this Dec 25, 2020
@liske liske added this to the apt-dater 1.0.4 milestone Dec 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants