Merge lp://staging/~jelmer/brz/authconfperm into lp://staging/brz
Proposed by
Jelmer Vernooij
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Jelmer Vernooij | ||||
Approved revision: | no longer in the source branch. | ||||
Merge reported by: | The Breezy Bot | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp://staging/~jelmer/brz/authconfperm | ||||
Merge into: | lp://staging/brz | ||||
Diff against target: |
144 lines (+64/-3) 4 files modified
breezy/config.py (+21/-1) breezy/help_topics/en/configuration.txt (+4/-0) breezy/tests/test_config.py (+34/-2) doc/en/release-notes/brz-3.0.txt (+5/-0) |
||||
To merge this branch: | bzr merge lp://staging/~jelmer/brz/authconfperm | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Packman | Approve | ||
Review via email:
|
Commit message
Use tight permissions on authentication.
Description of the change
When creating authentication.
When opening an existing authentication.
To post a comment you must log in.
General concept seems fine, though we still don't really want users putting passwords in there. Have some nits about the implementation inline.
Yet another change that makes me want to get to saner trace/warnings soon.