Skip to content

Commit c754232

Browse files
committed
AbstractRequestLoggingFilter.isIncludeHeaders() declared as protected
Issue: SPR-16881
1 parent 74fcdea commit c754232

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

Diff for: spring-web/src/main/java/org/springframework/web/filter/AbstractRequestLoggingFilter.java

+6-5
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,11 @@
4141
*
4242
* <p>Subclasses are passed the message to write to the log in the {@code beforeRequest} and
4343
* {@code afterRequest} methods. By default, only the URI of the request is logged. However,
44-
* setting the {@code includeQueryString} property to {@code true} will cause the query string
45-
* of the request to be included also. The payload (body) of the request can be logged via the
46-
* {@code includePayload} flag. Note that this will only log that which is read, which might
47-
* not be the entire payload.
44+
* setting the {@code includeQueryString} property to {@code true} will cause the query string of
45+
* the request to be included also; this can be further extended through {@code includeClientInfo}
46+
* and {@code includeHeaders}. The payload (body content) of the request can be logged via the
47+
* {@code includePayload} flag: Note that this will only log the part of the payload which has
48+
* actually been read, not necessarily the entire body of the request.
4849
*
4950
* <p>Prefixes and suffixes for the before and after messages can be configured using the
5051
* {@code beforeMessagePrefix}, {@code afterMessagePrefix}, {@code beforeMessageSuffix} and
@@ -137,7 +138,7 @@ public void setIncludeHeaders(boolean includeHeaders) {
137138
* Return whether the request headers should be included in the log message.
138139
* @since 4.3
139140
*/
140-
public boolean isIncludeHeaders() {
141+
protected boolean isIncludeHeaders() {
141142
return this.includeHeaders;
142143
}
143144

0 commit comments

Comments
 (0)