Fix race condition in some http conformance test

Make call to Exception.getMessage()/getReasonPhrase in synchronized
block to fix some flaky unit tests (e.g. testRequestContentCloseExceptionBeforeResponseWrite)

1 files changed, 2 insertions, 2 deletions

diff --git a/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/ServletResponseController.java b/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/ServletResponseController.java
index f90e0e8c856..a06d39ed016 100644
--- a/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/ServletResponseController.java
+++ b/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/ServletResponseController.java
@@ -101,9 +101,9 @@ public class ServletResponseController {
     public void trySendError(Throwable t) {
         final boolean responseWasCommitted;
         try {
-            String reasonPhrase = getReasonPhrase(t, developerMode);
-            int statusCode = getStatusCode(t);
             synchronized (monitor) {
+                String reasonPhrase = getReasonPhrase(t, developerMode);
+                int statusCode = getStatusCode(t);
                 responseWasCommitted = responseCommitted;
                 if (!responseCommitted) {
                     responseCommitted = true;