From ab07d0d86d8b64b4f7c88b45bc81f1eec22105db Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 10 Apr 2014 19:00:05 -0400
Subject: [PATCH] Preserve key index when re-adding an existing key

---
 src/main/java/com/gitblit/transport/ssh/gitblit/KeysDispatcher.java |   50 ++++++++++++++++++++++++++++++++++++++++++++++----
 1 files changed, 46 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/gitblit/transport/ssh/gitblit/KeysDispatcher.java b/src/main/java/com/gitblit/transport/ssh/gitblit/KeysDispatcher.java
index b42be17..5f508e6 100644
--- a/src/main/java/com/gitblit/transport/ssh/gitblit/KeysDispatcher.java
+++ b/src/main/java/com/gitblit/transport/ssh/gitblit/KeysDispatcher.java
@@ -33,6 +33,7 @@
 import com.gitblit.transport.ssh.commands.UsageExample;
 import com.gitblit.utils.FlipTable;
 import com.gitblit.utils.FlipTable.Borders;
+import com.google.common.base.Joiner;
 
 /**
  * The dispatcher and it's commands for SSH public key management.
@@ -49,6 +50,7 @@
 		register(user, RemoveKey.class);
 		register(user, ListKeys.class);
 		register(user, WhichKey.class);
+		register(user, CommentKey.class);
 	}
 
 	@CommandMetaData(name = "add", description = "Add an SSH public key to your account")
@@ -195,16 +197,56 @@
 			if (showRaw) {
 				stdout.println(key.getRawData());
 			} else {
-				asTable(key);
+				final String username = getContext().getClient().getUsername();
+				List<SshKey> keys = getContext().getGitblit().getPublicKeyManager().getKeys(username);
+				int index = 0;
+				for (int i = 0; i < keys.size(); i++) {
+					if (key.equals(keys.get(i))) {
+						index = i + 1;
+						break;
+					}
+				}
+				asTable(index, key);
 			}
 		}
 
-		protected void asTable(SshKey key) {
-			String[] headers = { "Fingerprint", "Comment", "Type" };
+		protected void asTable(int index, SshKey key) {
+			String[] headers = { "#", "Fingerprint", "Comment", "Type" };
 			Object[][] data = new Object[1][];
-			data[0] = new Object[] { key.getFingerprint(), key.getComment(), key.getAlgorithm() };
+			data[0] = new Object[] { index, key.getFingerprint(), key.getComment(), key.getAlgorithm() };
 
 			stdout.println(FlipTable.of(headers, data, Borders.BODY_HCOLS));
 		}
 	}
+
+	@CommandMetaData(name = "comment", description = "Set the comment for an SSH public key")
+	@UsageExample(syntax = "${cmd} 3 Home workstation", description = "Set the comment for key #3")
+	public static class CommentKey extends SshCommand {
+
+		@Argument(index = 0, metaVar = "INDEX", usage = "the key index", required = true)
+		private int index;
+
+		@Argument(index = 1, metaVar = "COMMENT", usage = "the new comment", required = true)
+		private List<String> values = new ArrayList<String>();
+
+		@Override
+		public void run() throws UnloggedFailure {
+			final String username = getContext().getClient().getUsername();
+			IPublicKeyManager keyManager = getContext().getGitblit().getPublicKeyManager();
+			List<SshKey> keys = keyManager.getKeys(username);
+			if (index > keys.size()) {
+				throw new UnloggedFailure(1,  "Invalid key index!");
+			}
+
+			String comment = Joiner.on(" ").join(values);
+			SshKey key = keys.get(index - 1);
+			key.setComment(comment);
+			if (keyManager.addKey(username, key)) {
+				stdout.println(String.format("Updated the comment for key #%d.", index));
+			} else {
+				throw new UnloggedFailure(1, String.format("Failed to update the comment for key #%d!", index));
+			}
+		}
+
+	}
 }

--
Gitblit v1.9.1