diff --git a/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletResponse.java b/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletResponse.java index 695c1277dcf2..6b12e65e6679 100644 --- a/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletResponse.java +++ b/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletResponse.java @@ -378,10 +378,10 @@ private String getCookieHeader(Cookie cookie) { buf.append("; Domain=").append(cookie.getDomain()); } int maxAge = cookie.getMaxAge(); + ZonedDateTime expires = (cookie instanceof MockCookie ? ((MockCookie) cookie).getExpires() : null); if (maxAge >= 0) { buf.append("; Max-Age=").append(maxAge); buf.append("; Expires="); - ZonedDateTime expires = (cookie instanceof MockCookie ? ((MockCookie) cookie).getExpires() : null); if (expires != null) { buf.append(expires.format(DateTimeFormatter.RFC_1123_DATE_TIME)); } @@ -391,6 +391,11 @@ private String getCookieHeader(Cookie cookie) { buf.append(headers.getFirst(HttpHeaders.EXPIRES)); } } + else if (expires != null) { + buf.append("; Expires="); + buf.append(expires.format(DateTimeFormatter.RFC_1123_DATE_TIME)); + } + if (cookie.getSecure()) { buf.append("; Secure"); diff --git a/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java b/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java index 08fda3f72372..c7317fb8c319 100644 --- a/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java +++ b/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java @@ -475,6 +475,17 @@ void addCookieHeaderWithZeroExpiresAttribute() { assertThat(header).startsWith("SESSION=123; Path=/; Max-Age=100; Expires="); } + /** + * @since 5.1.12 + */ + @Test + void addCookieHeaderWithOnlyExpiresAttribute() { + String cookieValue = "SESSION=123; Path=/; Expires=Tue, 8 Oct 2019 19:50:00 GMT"; + response.addHeader(SET_COOKIE, cookieValue); + assertNumCookies(1); + assertThat(response.getHeader(SET_COOKIE)).isEqualTo(cookieValue); + } + @Test void addCookie() { MockCookie mockCookie = new MockCookie("SESSION", "123");