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.

Sign up for GitHub

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

Jump to bottom

fix custom file attributes for public keys #257

Merged
merged 1 commit into from
Jul 20, 2021
Merged

fix custom file attributes for public keys #257

merged 1 commit into from
Jul 20, 2021

Commits on Jul 20, 2021

  1. fix custom file attributes for public keys 

    Use of the confusingly-named _permissions_changed() on both
sides of an `or` was resulting in the second invocation not
being reached if the first invocation returned True, which it
does any time it applied custom attributes to the private key.
As a result, custom file attributes were only ever being
applied to the private key (except in one specific case)

This is fixed by explicitly updating attributes of both files
before checking if changes have been made.

Signed-off-by: Charlie Wheeler-Robinson <[email protected]>
    @crwr45
    crwr45 committed Jul 20, 2021
    Configuration menu
    Copy the full SHA
    e114e75 View commit details
    Browse the repository at this point in the history