Skip to content

Commit ebbbff4

Browse files
committed
refactor(CspController): don't use @slf4j annotation from Lombok.
This makes code more obvious.
1 parent 99ab8f2 commit ebbbff4

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/main/java/ru/mystamps/web/feature/site/CspController.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,9 @@
1717
*/
1818
package ru.mystamps.web.feature.site;
1919

20-
import lombok.extern.slf4j.Slf4j;
2120
import org.apache.commons.lang3.StringUtils;
21+
import org.slf4j.Logger;
22+
import org.slf4j.LoggerFactory;
2223
import org.springframework.http.HttpStatus;
2324
import org.springframework.web.bind.annotation.PostMapping;
2425
import org.springframework.web.bind.annotation.RequestBody;
@@ -30,8 +31,9 @@
3031
import java.util.regex.Pattern;
3132

3233
@RestController
33-
@Slf4j
3434
public class CspController {
35+
private static final Logger LOG = LoggerFactory.getLogger(CspController.class);
36+
3537
private static final String UNKNOWN = "<unknown>";
3638

3739
private static final Pattern ORIGINAL_POLICY_PATTERN = Pattern.compile(
@@ -46,11 +48,11 @@ public void handleReport(
4648
@RequestHeader(name = "user-agent", defaultValue = UNKNOWN) String userAgent) {
4749

4850
String ip = StringUtils.defaultString(request.getRemoteAddr(), UNKNOWN);
49-
log.warn("CSP report from IP: {}, user agent: {}", ip, userAgent);
51+
LOG.warn("CSP report from IP: {}, user agent: {}", ip, userAgent);
5052

5153
// Omit "original-policy" as it quite long and useless most of the time
5254
String report = ORIGINAL_POLICY_PATTERN.matcher(body).replaceFirst(StringUtils.EMPTY);
53-
log.warn(report);
55+
LOG.warn(report);
5456
}
5557

5658
}

0 commit comments

Comments
 (0)