Propagate http status code when http upgrade request failed

[email protected]

Change-Id: I547d19da251bdea0c259d3896e6e333f9afd0bca
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/384522
Auto-Submit: Thomas Guerin <[email protected]>
Reviewed-by: Brian Quinlan <[email protected]>
Commit-Queue: Alexander Aprelev <[email protected]>
Reviewed-by: Alexander Aprelev <[email protected]>

sdk/lib/_http/websocket.dart

@@ -407,8 +407,14 @@ abstract class WebSocket
 
 class WebSocketException implements IOException {
   final String message;
+  final int? httpStatusCode;
 
-  const WebSocketException([this.message = ""]);
+  const WebSocketException([this.message = "", this.httpStatusCode]);
 
-  String toString() => "WebSocketException: $message";
+  String toString() {
+    if (httpStatusCode != null) {
+      return "WebSocketException: $message, HTTP status code: $httpStatusCode";
+    }
+    return "WebSocketException: $message";
+  }
 }

sdk/lib/_http/websocket_impl.dart

@@ -1046,13 +1046,13 @@ class _WebSocketImpl extends Stream with _ServiceObject implements WebSocket {
 
       return request.close();
     }).then((response) {
-      Future<WebSocket> error(String message) {
+      Future<WebSocket> error(String message, [int? httpStatusCode]) {
         // Flush data.
         response.detachSocket().then((socket) {
           socket.destroy();
         });
         return Future<WebSocket>.error(
-            WebSocketException(message), callerStackTrace);
+            WebSocketException(message, httpStatusCode), callerStackTrace);
       }
 
       var connectionHeader = response.headers[HttpHeaders.connectionHeader];
@@ -1061,7 +1061,8 @@ class _WebSocketImpl extends Stream with _ServiceObject implements WebSocket {
           !connectionHeader.any((value) => value.toLowerCase() == "upgrade") ||
           response.headers.value(HttpHeaders.upgradeHeader)!.toLowerCase() !=
               "websocket") {
-        return error("Connection to '$uri' was not upgraded to websocket");
+        return error("Connection to '$uri' was not upgraded to websocket",
+            response.statusCode);
       }
       String? accept = response.headers.value("Sec-WebSocket-Accept");
       if (accept == null) {

tests/standalone/io/web_socket_test.dart

@@ -300,6 +300,7 @@ class SecurityConfiguration {
       });
 
       Future<WebSocket?>.value(createClient(server.port)).catchError((error) {
+        Expect.equals(HttpStatus.notFound, error.httpStatusCode);
         server.close();
       });
     });