Skip to content

Commit 4557afd

Browse files
Improve comments for review
1 parent a0b0935 commit 4557afd

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

cf-java-logging-support-servlet/src/main/java/com/sap/hcp/cf/logging/servlet/dynlog/DynamicLogLevelProcessor.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ public class DynamicLogLevelProcessor {
3232
* {@link DynamicLogLevelProcessor#DynamicLogLevelProcessor(RSAPublicKey)}
3333
* instead.
3434
* @param dynLogConfig
35-
* then {@link DynLogConfiguration} to read the public RSA key
36-
* for JWT validation from.
35+
* the {@link DynLogConfiguration} to read the public RSA key for
36+
* JWT validation from.
3737
*/
3838
@Deprecated
3939
public DynamicLogLevelProcessor(DynLogConfiguration dynLogConfig) {
@@ -90,8 +90,9 @@ protected void processJWT(DecodedJWT jwt) throws DynamicLogLevelException {
9090
}
9191

9292
/**
93-
* Removes the MDC fields for dynamic log levels. This needs to be called to
94-
* remove the changed log level configuration.
93+
* Resets the current dynamic log level configuration by removing the
94+
* corresponding fields from the MDC. This needs to be called to remove the
95+
* changed log level configuration.
9596
*/
9697
public void removeDynamicLogLevelFromMDC() {
9798
MDC.remove(DynamicLogLevelHelper.MDC_DYNAMIC_LOG_LEVEL_KEY);

cf-java-logging-support-servlet/src/main/java/com/sap/hcp/cf/logging/servlet/filter/RequestLoggingFilter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ private static ConcurrentInitializer<DynLogEnvironment> createDefaultDynLogEnvir
9696
public RequestLoggingFilter(ConcurrentInitializer<DynLogEnvironment> dynLogEnvironment) {
9797
this(createDefaultRequestRecordFactory(), dynLogEnvironment);
9898
}
99-
99+
100100
public RequestLoggingFilter(RequestRecordFactory requestRecordFactory,
101101
ConcurrentInitializer<DynLogEnvironment> dynLogEnvironment) {
102102
this.requestRecordFactory = requestRecordFactory;

0 commit comments

Comments
 (0)