Page MenuHomePhabricator

When administrators revoke SSH keys, don't include a "security warning" in the mail
ClosedPublic

Authored by epriestley on Jan 22 2018, 1:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 2:26 AM
Unknown Object (File)
Thu, May 2, 1:23 PM
Unknown Object (File)
Wed, May 1, 6:06 PM
Unknown Object (File)
Mon, Apr 29, 1:56 PM
Unknown Object (File)
Wed, Apr 24, 10:02 PM
Unknown Object (File)
Sun, Apr 21, 4:07 PM
Unknown Object (File)
Fri, Apr 19, 7:07 PM
Unknown Object (File)
Fri, Apr 19, 1:10 PM
Subscribers
None

Details

Summary

Depends on D18906. Ref T13043. When SSH keys are edited, we normally include a warning that if you don't recognize the activity you might have problems in the mail body.

Currently, this warning is also shown for revocations with bin/auth revoke --type ssh. However, these revocations are safe (revocations are generally not dangerous anyway) and almost certainly legitimate and administrative, so don't warn users about them.

Test Plan
  • Created and revoked a key.
  • Creation mail still had warning; revocation mail no longer did.

Diff Detail

Repository
rP Phabricator
Branch
revoke17
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19110
Build 25800: Run Core Tests
Build 25799: arc lint + arc unit