Skip to content

Commit 444f116

Browse files
authored
remove secretKey from extractor log (#117)
* remove secretKey from extractor log * trycatch potential exception * trycatch potential exception
1 parent c14557f commit 444f116

File tree

2 files changed

+26
-9
lines changed

2 files changed

+26
-9
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
2121
disable and enable triggers.
2222

2323
### Fixed
24+
- Remove secretKey from extractor log.
2425
- Ignore the `update` field when posting to `/api/extractors`. [#89](https://github.com/clowder-framework/clowder/issues/89)
2526
- Search results were hardcoded to be in batches of 2.
2627

app/services/GraylogService.scala

Lines changed: 25 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -48,17 +48,33 @@ class GraylogService @Inject() extends LogService {
4848
val responseJsonValue = JsonUtil.parseJSON(responseJson).asInstanceOf[java.util.LinkedHashMap[String, java.util.LinkedList[java.util.LinkedHashMap[String, String]]]]
4949
val messagesjsonValue = responseJsonValue.get("messages")
5050
messagesjsonValue.foreach(elem => {
51-
val elemMap = elem.asInstanceOf[java.util.HashMap[String, java.util.HashMap[String, String]]]
52-
val message: String = elemMap.get("message").get("message")
53-
val keyIndex: Int = message.indexOf("key=")
54-
keyIndex match {
55-
case -1 => {
56-
logs = logs :+ message
51+
var message: String = ""
52+
try {
53+
val elemMap = elem.asInstanceOf[java.util.HashMap[String, java.util.HashMap[String, String]]]
54+
message = elemMap.get("message").get("message")
55+
56+
val secretKeyIndex: Int = message.indexOf("'secretKey':")
57+
val secretKeyLen: Int = "'secretKey':".length()
58+
if (secretKeyIndex >= 0) {
59+
val prev: String = message.substring(0, secretKeyIndex)
60+
val sub: String = message.substring(secretKeyIndex + secretKeyLen)
61+
val secretValueIndex: Int = sub.indexOf("',")
62+
val suffix: String = sub.substring(secretValueIndex + "',".length())
63+
message = prev + suffix
5764
}
58-
case _ => {
59-
val filterMessage: String = message.substring(0, keyIndex)
60-
logs = logs :+ filterMessage
65+
66+
val keyIndex: Int = message.indexOf("key=")
67+
keyIndex match {
68+
case -1 => {
69+
logs = logs :+ message
70+
}
71+
case _ => {
72+
val filterMessage: String = message.substring(0, keyIndex)
73+
logs = logs :+ filterMessage
74+
}
6175
}
76+
} catch {
77+
case e: Exception => Logger.warn(s"Exception: '$message'")
6278
}
6379
}
6480
)

0 commit comments

Comments
 (0)