Skip to content

Commit cb21862

Browse files
author
Daan Hoogland
committed
adjusted regex to be in line with sessionkey format
1 parent 16177aa commit cb21862

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

utils/src/main/java/com/cloud/utils/StringUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ public static String getMaskedPasswordForDisplay(final String password) {
150150

151151
private static final Pattern REGEX_PASSWORD_DETAILS_INDEX = Pattern.compile("details(\\[|%5B)\\d*(\\]|%5D)");
152152

153-
private static final Pattern REGEX_SESSION_KEY = Pattern.compile("sessionkey=(\\p{Alnum}*)");
153+
private static final Pattern REGEX_SESSION_KEY = Pattern.compile("sessionkey=[A-Za-z0-9_-]+");
154154

155155
private static final Pattern REGEX_REDUNDANT_AND = Pattern.compile("(&|%26)(&|%26)+");
156156

utils/src/test/java/com/cloud/utils/StringUtilsTest.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,13 @@ public void testCleanSecretkeyFromRequestString() {
240240
assertEquals(result, expected);
241241
}
242242

243+
@Test
244+
public void testCleanSessionkeyFromAccessLogString() {
245+
final String input = "GET /client/api/?managementserverid=cad7010f-216f-48cb-af11-280588863c4e&command=readyForShutdown&response=json&sessionkey=-FrgnKy6pj-JB4BI2sXqo HTTP/1.1\" 200 180 \"-\" \"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/18.5 Safari/605.1.15 Ddg/18.5";
246+
final String expected = "GET /client/api/?managementserverid=cad7010f-216f-48cb-af11-280588863c4e&command=readyForShutdown&response=json& HTTP/1.1\" 200 180 \"-\" \"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/18.5 Safari/605.1.15 Ddg/18.5";
247+
final String result = StringUtils.cleanString(input);
248+
assertEquals(expected, result);
249+
}
243250
@Test
244251
public void testCleanSessionkeyFromRequestJsonString() {
245252
final String input = "{id=64b5e71d-2ae8-11ef-9466-1e00c400042b, showicon=true, command=listUsers, response=json, sessionkey=lXfAicKQXPBzt7KjLx6DwVfcOuA}";

0 commit comments

Comments
 (0)