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
F19939973: D18907.diff
Tue, Apr 14, 8:30 PM
F19936677: D18907.diff
Sun, Apr 12, 3:49 PM
F19927405: D18907.id45368.diff
Tue, Apr 7, 10:32 PM
F19911917: D18907.diff
Thu, Apr 2, 3:53 PM
F19559076: D18907.diff
Jan 31 2026, 12:32 AM
F19241101: D18907.id45368.diff
Dec 21 2025, 10:19 PM
F19000154: D18907.diff
Nov 20 2025, 11:03 PM
F18811564: D18907.id45368.diff
Oct 20 2025, 3:54 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable