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

Use MAJOR:MINOR versioning scheme for the gui protocol #621

Closed
marmarek opened this Issue Mar 8, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@marmarek
Member

marmarek commented Mar 8, 2015

Reported by joanna on 12 Jul 2012 13:11 UTC
MAJOR number should be increased when (and only when) the change of the
GUI protocol requires rebuild of all the gui agents and the gui daemon.

MINOR number should be increased when the change of the GUI protocol
requires only gui daemon and select agents rebuild.

It should be very rare situation when we would need to increase the MAJOR number.

A gui daemon, seeing that it talks with a gui agent of wrong MAJOR number, should display a message to the user informing about the need to upgrade the agent. This should, however, not be required for proper operation with a gui agent with a smaller MINOR number (and the same MAJOR), as the guid.

Of course, guid should always be rebuild to have the highest MINOR number, as otherwise it would refuse to talk with an agent that has a newer MINOR than the guid.

This doesn't present any security risk, because the guid always treats messages from gui agents as untrusted.

Migrated-From: https://wiki.qubes-os.org/ticket/621

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Modified by marmarek on 12 Jul 2012 14:49 UTC

Member

marmarek commented Mar 8, 2015

Modified by marmarek on 12 Jul 2012 14:49 UTC

@marmarek marmarek added this to the Release 1 milestone Mar 8, 2015

@marmarek marmarek self-assigned this Mar 8, 2015

@marmarek

This comment has been minimized.

Show comment
Hide comment

@marmarek marmarek closed this Mar 8, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment