appium-webdriveragent 16.6.0 → 16.7.0

package/WebDriverAgentLib/Info.plist

@@ -15,11 +15,11 @@
     <key>CFBundlePackageType</key>
     <string>FMWK</string>
     <key>CFBundleShortVersionString</key>
-    <string>16.6.0</string>
+    <string>16.7.0</string>
     <key>CFBundleSignature</key>
     <string>????</string>
     <key>CFBundleVersion</key>
-    <string>16.6.0</string>
+    <string>16.7.0</string>
     <key>NSPrincipalClass</key>
     <string/>
   </dict>

package/WebDriverAgentLib/Routing/{WatchOS/FBWatchHTTPServer.h → FBHTTPServer.h}

@@ -6,13 +6,11 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-// RoutingHTTPServer/CocoaHTTPServer need BSD sockets (via GCDAsyncSocket), which watchOS
-// forbids - see FBTCPSocket.h/.m. This is a minimal HTTP/1.1 server on top of the watchOS
-// FBTCPSocket, mirroring just enough of RoutingHTTPServer's API for FBWebServer.m to swap
-// servers with a single #if TARGET_OS_WATCH.
+// A minimal HTTP/1.1 server on top of FBTCPSocket (Network.framework-backed on every platform,
+// since watchOS forbids BSD sockets outright - see FBTCPSocket.h/.m).
 //
 // No chunked encoding, range requests, or pipelining - just request line + headers +
-// Content-Length body, and ":param" path matching like RoutingHTTPServer.m.
+// Content-Length body, and ":param" path matching.
 
 @import Foundation;
 
@@ -21,7 +19,7 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
-@interface FBWatchHTTPServer : NSObject
+@interface FBHTTPServer : NSObject
 
 /*! The port the server is (or will be) listening on */
 @property (nonatomic) uint16_t port;
@@ -40,9 +38,15 @@ NS_ASSUME_NONNULL_BEGIN
  */
 - (void)setDefaultHeader:(NSString *)field value:(NSString *)value;
 
+/**
+ Sets the local IP address to bind the listener to. Must be called before -start:. Pass nil (the
+ default) to listen on all interfaces.
+ */
+- (void)setInterface:(nullable NSString *)interface;
+
 /**
  Registers a route handler for the given HTTP method and path pattern (":param" segments are
- captured into the request's `params`, matching RoutingHTTPServer's convention).
+ captured into the request's `params`).
  */
 - (void)handleMethod:(NSString *)method
             withPath:(NSString *)path

package/WebDriverAgentLib/Routing/{WatchOS/FBWatchHTTPServer.m → FBHTTPServer.m}

@@ -6,7 +6,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-#import "FBWatchHTTPServer.h"
+#import "FBHTTPServer.h"
 
 #import "FBConfiguration.h"
 #import "FBTCPSocket.h"
@@ -28,29 +28,30 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
   return (NSData * _Nonnull)[string dataUsingEncoding:NSUTF8StringEncoding];
 }
 
-@interface FBWatchHTTPRoute : NSObject
+@interface FBHTTPRoute : NSObject
 @property (nonatomic, copy) NSString *verb;
 @property (nonatomic, strong) NSRegularExpression *regex;
 @property (nonatomic, copy, nullable) NSArray<NSString *> *keys;
 @property (nonatomic, copy) void (^block)(RouteRequest *request, RouteResponse *response);
 @end
 
-@implementation FBWatchHTTPRoute
+@implementation FBHTTPRoute
 @end
 
 
-@interface FBWatchHTTPServer () <FBTCPSocketDelegate>
+@interface FBHTTPServer () <FBTCPSocketDelegate>
 
 @property (nonatomic, nullable, strong) FBTCPSocket *socket;
-@property (nonatomic, strong) NSMutableArray<FBWatchHTTPRoute *> *routes;
+@property (nonatomic, strong) NSMutableArray<FBHTTPRoute *> *routes;
 @property (nonatomic, strong) NSMutableDictionary<NSString *, NSString *> *defaultHeaders;
 @property (nonatomic, nullable) dispatch_queue_t routeQueue;
+@property (nonatomic, copy, nullable) NSString *interface;
 // nw_connection_t isn't NSCopying, so it can't be an NSDictionary key - use NSMapTable instead.
 @property (nonatomic, strong) NSMapTable<id, NSMutableData *> *connectionBuffers;
 
 @end
 
-@implementation FBWatchHTTPServer
+@implementation FBHTTPServer
 
 - (instancetype)init
 {
@@ -73,15 +74,20 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
   self.defaultHeaders[field] = value;
 }
 
+- (void)setInterface:(nullable NSString *)interface
+{
+  _interface = interface.copy;
+}
+
 #pragma mark - Route registration
 
-- (FBWatchHTTPRoute *)compiledRouteWithPath:(NSString *)path
+- (FBHTTPRoute *)compiledRouteWithPath:(NSString *)path
 {
-  FBWatchHTTPRoute *route = [FBWatchHTTPRoute new];
+  FBHTTPRoute *route = [FBHTTPRoute new];
   NSMutableArray<NSString *> *keys = [NSMutableArray array];
 
   // Escape regex-significant characters before substituting :param placeholders, like
-  // RoutingHTTPServer.m does.
+  // RoutingHTTPServer.m used to.
   NSRegularExpression *escapeRegex = [NSRegularExpression regularExpressionWithPattern:@"[.+()]"
                                                                                 options:(NSRegularExpressionOptions)0
                                                                                   error:nil];
@@ -126,7 +132,7 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
             withPath:(NSString *)path
                block:(void (^)(RouteRequest *request, RouteResponse *response))block
 {
-  FBWatchHTTPRoute *route = [self compiledRouteWithPath:path];
+  FBHTTPRoute *route = [self compiledRouteWithPath:path];
   route.verb = method.uppercaseString;
   route.block = block;
   [self.routes addObject:route];
@@ -142,6 +148,7 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
 - (BOOL)start:(NSError **)error
 {
   FBTCPSocket *socket = [[FBTCPSocket alloc] initWithPort:self.port];
+  socket.interface = self.interface;
   socket.delegate = self;
   if (![socket startWithError:error]) {
     return NO;
@@ -239,8 +246,8 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
 
     NSUInteger contentLength = (NSUInteger)requestHeaders[@"content-length"].integerValue;
     if (contentLength > FBConfiguration.sharedInstance.httpRequestBodySizeLimit) {
-      // Mirrors FBHTTPConnection's maxRequestBodySize enforcement on iOS/tvOS. Closes the
-      // connection after responding, since the rest of the oversized body is still incoming.
+      // Mirrors CocoaHTTPServer's maxRequestBodySize enforcement. Closes the connection after
+      // responding, since the rest of the oversized body is still incoming.
       RouteResponse *tooLarge = [RouteResponse new];
       tooLarge.statusCode = kHTTPStatusCodeRequestEntityTooLarge;
       [tooLarge respondWithString:@"Request Entity Too Large"];
@@ -269,7 +276,7 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
   NSURLComponents *requestTarget = [NSURLComponents componentsWithString:pathAndQuery];
   NSString *path = requestTarget.path ?: pathAndQuery;
 
-  for (FBWatchHTTPRoute *route in self.routes) {
+  for (FBHTTPRoute *route in self.routes) {
     if (![route.verb isEqualToString:method]) {
       continue;
     }
@@ -367,6 +374,12 @@ static NSData * _Nonnull FBUTF8Data(NSString *string)
     return @"Bad Request";
   } else if (kHTTPStatusCodeNotFound == statusCode) {
     return @"Not Found";
+  } else if (kHTTPStatusCodeMethodNotAllowed == statusCode) {
+    return @"Method Not Allowed";
+  } else if (kHTTPStatusCodeRequestTimeout == statusCode) {
+    return @"Request Timeout";
+  } else if (kHTTPStatusCodeRequestEntityTooLarge == statusCode) {
+    return @"Request Entity Too Large";
   } else if (kHTTPStatusCodeInternalServerError == statusCode) {
     return @"Internal Server Error";
   }

package/WebDriverAgentLib/Routing/FBTCPSocket.h

@@ -6,28 +6,18 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-// TARGET_OS_WATCH must be defined before the #if below runs, or (on some older Xcode/SDK
-// toolchains) it silently evaluates as undefined/false here despite being true for the rest of
-// the translation unit - which desyncs this file's declarations from FBTCPSocket.m's own
-// #if TARGET_OS_WATCH branch.
-#import <TargetConditionals.h>
-
-#if TARGET_OS_WATCH
 @import Foundation;
 // A textual import, not `@import Network;` - older Xcode/watchOS SDK combinations (verified:
 // Xcode 15.4/watchOS 10.5) fail to expose nw_listener_t/nw_connection_t and friends through the
 // Network module map on watchOS, even though the underlying API has existed since watchOS 5.0.
+// Kept unconditional (rather than gated to watchOS) since it also builds cleanly on iOS/tvOS.
 #import <Network/Network.h>
-#else
-#import "GCDAsyncSocket.h"
-#endif
 
 NS_ASSUME_NONNULL_BEGIN
 
-#if TARGET_OS_WATCH
-
-// watchOS forbids BSD sockets, so this is backed by Network.framework instead of GCDAsyncSocket -
-// hence a differently-shaped, push-style delegate protocol.
+// Backed by Network.framework rather than BSD sockets on every platform, since watchOS forbids
+// BSD sockets outright and there is no reason to keep a second, socket-based implementation
+// around just for iOS/tvOS.
 @protocol FBTCPSocketDelegate <NSObject>
 
 /**
@@ -54,35 +44,6 @@ NS_ASSUME_NONNULL_BEGIN
 
 @end
 
-#else
-
-@protocol FBTCPSocketDelegate
-
-/**
- The callback which is fired on new TCP client connection
-
- @param newClient The newly connected socket
- */
-- (void)didClientConnect:(GCDAsyncSocket *)newClient;
-
-/**
- The callback which is fired when the TCP server receives a data from a connected client
-
- @param client The client, which sent the data
-*/
-- (void)didClientSendData:(GCDAsyncSocket *)client;
-
-/**
- The callback which is fired when TCP client disconnects
-
- @param client The actual diconnected client
- */
-- (void)didClientDisconnect:(GCDAsyncSocket *)client;
-
-@end
-
-#endif
-
 
 @interface FBTCPSocket : NSObject
 
@@ -99,6 +60,12 @@ NS_ASSUME_NONNULL_BEGIN
  */
 @property (nonatomic, readonly) uint16_t port;
 
+/**
+ The local IP address to bind the listener to, or nil to listen on all interfaces. Must be set
+ before -startWithError: is called.
+ */
+@property (nonatomic, copy, nullable) NSString *interface;
+
 /**
  Creates TCP socket isntance which is going to be started on the specified port
 
@@ -120,7 +87,6 @@ NS_ASSUME_NONNULL_BEGIN
  */
 - (void)stop;
 
-#if TARGET_OS_WATCH
 /**
  Writes data to the given connected client
 
@@ -139,7 +105,6 @@ NS_ASSUME_NONNULL_BEGIN
  @param completion Called once the send attempt finishes
  */
 - (void)writeData:(NSData *)data toClient:(nw_connection_t)client completion:(nullable void (^)(void))completion;
-#endif
 
 @end
 

package/WebDriverAgentLib/Routing/FBTCPSocket.m

@@ -8,8 +8,6 @@
 
 #import "FBTCPSocket.h"
 
-#if TARGET_OS_WATCH
-
 @interface FBTCPSocket()
 @property (readonly, nonatomic) dispatch_queue_t socketQueue;
 @property (nullable, nonatomic) nw_listener_t listener;
@@ -37,7 +35,18 @@
   NSString *portString = [NSString stringWithFormat:@"%u", (unsigned int)self.port];
   // portString is always valid UTF8; -UTF8String is just declared nullable in general.
   const char * _Nonnull portCString = (const char * _Nonnull)portString.UTF8String;
-  nw_listener_t listener = nw_listener_create_with_port(portCString, parameters);
+
+  nw_listener_t listener;
+  if (nil != self.interface) {
+    const char * _Nonnull interfaceCString = (const char * _Nonnull)((NSString * _Nonnull)self.interface).UTF8String;
+    nw_endpoint_t localEndpoint = nw_endpoint_create_host(interfaceCString, portCString);
+    nw_parameters_set_local_endpoint(parameters, localEndpoint);
+    // The port is already encoded in localEndpoint above - do not also pass it to
+    // nw_listener_create_with_port, which would be ambiguous.
+    listener = nw_listener_create(parameters);
+  } else {
+    listener = nw_listener_create_with_port(portCString, parameters);
+  }
   if (nil == listener) {
     if (error) {
       *error = [NSError errorWithDomain:@"FBTCPSocket"
@@ -206,94 +215,3 @@
 }
 
 @end
-
-#else
-
-@interface FBTCPSocket()
-@property (readonly, nonatomic) dispatch_queue_t socketQueue;
-@property (readonly, nonatomic) GCDAsyncSocket *listeningSocket;
-@property (readonly, nonatomic) NSMutableArray *connectedClients;
-@end
-
-
-@interface FBTCPSocket(AsyncSocket) <GCDAsyncSocketDelegate>
-
-@end
-
-
-@implementation FBTCPSocket
-
-- (instancetype)initWithPort:(uint16_t)port
-{
-  if ((self = [super init])) {
-    _socketQueue = dispatch_queue_create("socketQueue", NULL);
-    _listeningSocket = [[GCDAsyncSocket alloc] initWithDelegate:self delegateQueue:_socketQueue];
-    _connectedClients = [[NSMutableArray alloc] initWithCapacity:1];
-    _port = port;
-    _delegate = nil;
-  }
-  return self;
-}
-
-- (BOOL)startWithError:(NSError **)error
-{
-  if (![self.listeningSocket acceptOnPort:self.port error:error]) {
-    return NO;
-  }
-
-  _port = self.listeningSocket.localPort;
-  return YES;
-}
-
-- (void)stop
-{
-  @synchronized(self.connectedClients) {
-    NSArray *clients = self.connectedClients.copy;
-    [self.connectedClients removeAllObjects];
-    for (GCDAsyncSocket *client in clients) {
-      [client disconnect];
-    }
-  }
-
-  self.delegate = nil;
-  [self.listeningSocket disconnect];
-}
-
-@end
-
-
-@implementation FBTCPSocket(AsyncSocket)
-
-- (void)socket:(GCDAsyncSocket *)sock didAcceptNewSocket:(GCDAsyncSocket *)newSocket
-{
-  @synchronized(self.connectedClients) {
-    [self.connectedClients addObject:newSocket];
-  }
-  id<FBTCPSocketDelegate> delegate = self.delegate;
-  if (nil != delegate) {
-    [delegate didClientConnect:newSocket];
-  }
-}
-
-- (void)socket:(GCDAsyncSocket *)sock didReadData:(NSData *)data withTag:(long)tag
-{
-  id<FBTCPSocketDelegate> delegate = self.delegate;
-  if (nil != delegate) {
-    [delegate didClientSendData:sock];
-  }
-}
-
-- (void)socketDidDisconnect:(GCDAsyncSocket *)sock withError:(NSError *)err
-{
-  @synchronized(self.connectedClients) {
-    [self.connectedClients removeObject:sock];
-  }
-  id<FBTCPSocketDelegate> delegate = self.delegate;
-  if (nil != delegate) {
-    [delegate didClientDisconnect:sock];
-  }
-}
-
-@end
-
-#endif

package/WebDriverAgentLib/Routing/FBWebServer.h

@@ -8,7 +8,7 @@
 
 #import <Foundation/Foundation.h>
 
-@class RouteResponse, RoutingHTTPServer, FBExceptionHandler;
+@class RouteResponse, FBExceptionHandler;
 @protocol FBWebServerDelegate;
 
 NS_ASSUME_NONNULL_BEGIN

package/WebDriverAgentLib/Routing/FBWebServer.m

@@ -8,12 +8,7 @@
 
 #import "FBWebServer.h"
 
-#if TARGET_OS_WATCH
-#import "FBWatchHTTPServer.h"
-#else
-#import "RoutingConnection.h"
-#import "RoutingHTTPServer.h"
-#endif
+#import "FBHTTPServer.h"
 #import "FBMjpegServer.h"
 #import "FBTCPSocket.h"
 
@@ -32,34 +27,9 @@
 static NSString *const FBServerURLBeginMarker = @"ServerURLHere->";
 static NSString *const FBServerURLEndMarker = @"<-ServerURLHere";
 
-#if !TARGET_OS_WATCH
-@interface FBHTTPConnection : RoutingConnection
-@end
-
-@implementation FBHTTPConnection
-
-- (void)handleResourceNotFound
-{
-  [FBLogger logFmt:@"Received request for %@ which we do not handle", self.requestURI];
-  [super handleResourceNotFound];
-}
-
-- (UInt64)maxRequestBodySize
-{
-  return FBConfiguration.sharedInstance.httpRequestBodySizeLimit;
-}
-
-@end
-#endif
-
-
 @interface FBWebServer ()
 @property (nonatomic, strong) FBExceptionHandler *exceptionHandler;
-#if TARGET_OS_WATCH
-@property (nonatomic, strong) FBWatchHTTPServer *server;
-#else
-@property (nonatomic, strong) RoutingHTTPServer *server;
-#endif
+@property (nonatomic, strong) FBHTTPServer *server;
 @property (nonatomic, nullable) FBTCPSocket *screenshotsBroadcaster;
 @property (nonatomic, nullable, strong) FBMjpegServer *mjpegServer;
 @property (atomic, assign) BOOL keepAlive;
@@ -104,30 +74,21 @@ static NSString *const FBServerURLEndMarker = @"<-ServerURLHere";
 
 - (BOOL)startHTTPServer
 {
-#if TARGET_OS_WATCH
-  self.server = [[FBWatchHTTPServer alloc] init];
-#else
-  self.server = [[RoutingHTTPServer alloc] init];
-#endif
+  self.server = [[FBHTTPServer alloc] init];
   [self.server setRouteQueue:dispatch_get_main_queue()];
   [self.server setDefaultHeader:@"Server" value:@"WebDriverAgent/1.0"];
   [self.server setDefaultHeader:@"Access-Control-Allow-Origin" value:@"*"];
   [self.server setDefaultHeader:@"Access-Control-Allow-Headers" value:@"Content-Type, X-Requested-With"];
-#if !TARGET_OS_WATCH
-  [self.server setConnectionClass:[FBHTTPConnection self]];
-#endif
 
   [self registerRouteHandlers:[self.class collectCommandHandlerClasses]];
   [self registerServerKeyRouteHandlers];
 
   NSRange serverPortRange = FBConfiguration.sharedInstance.bindingPortRange;
   NSString *bindingIP = FBConfiguration.sharedInstance.bindingIPAddress;
-#if !TARGET_OS_WATCH
   if (bindingIP != nil) {
     [self.server setInterface:bindingIP];
     [FBLogger logFmt:@"Using custom binding IP address: %@", bindingIP];
   }
-#endif
 
   NSError *error;
   BOOL serverStarted = NO;
@@ -165,9 +126,7 @@ static NSString *const FBServerURLEndMarker = @"<-ServerURLHere";
   self.mjpegServer = [[FBMjpegServer alloc] init];
   self.screenshotsBroadcaster = [[FBTCPSocket alloc]
                                  initWithPort:(uint16_t)FBConfiguration.sharedInstance.mjpegServerPort];
-#if TARGET_OS_WATCH
   self.mjpegServer.socket = self.screenshotsBroadcaster;
-#endif
   self.screenshotsBroadcaster.delegate = self.mjpegServer;
   NSError *error;
   if (![self.screenshotsBroadcaster startWithError:&error]) {
@@ -220,11 +179,7 @@ static NSString *const FBServerURLEndMarker = @"<-ServerURLHere";
   self.keepAlive = NO;
 }
 
-#if TARGET_OS_WATCH
-- (BOOL)attemptToStartServer:(FBWatchHTTPServer *)server onPort:(NSInteger)port withError:(NSError **)error
-#else
-- (BOOL)attemptToStartServer:(RoutingHTTPServer *)server onPort:(NSInteger)port withError:(NSError **)error
-#endif
+- (BOOL)attemptToStartServer:(FBHTTPServer *)server onPort:(NSInteger)port withError:(NSError **)error
 {
   server.port = (UInt16)port;
   NSError *innerError = nil;

package/WebDriverAgentLib/Routing/{WatchOS/RouteRequest.h → RouteRequest.h}

@@ -6,10 +6,8 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-// Minimal, watchOS-only stand-in for Vendor/RoutingHTTPServer/RouteRequest.h, which is not
-// available on watchOS because RoutingHTTPServer/CocoaHTTPServer/CocoaAsyncSocket cannot be
-// built there (see FBWatchHTTPServer.h). Exposes just the surface FBWebServer's route blocks
-// and FBRoute.decorateRequest: read.
+// A minimal request value type, exposing just the surface FBWebServer's route blocks and
+// FBRoute.decorateRequest: read.
 
 @import Foundation;
 

package/WebDriverAgentLib/Routing/{WatchOS/RouteResponse.h → RouteResponse.h}

@@ -6,8 +6,8 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-// Minimal, watchOS-only stand-in for Vendor/RoutingHTTPServer/RouteResponse.h, reproducing just
-// what FBRoute.m/FBResponseJSONPayload.m call on it. See FBWatchHTTPServer.h.
+// A minimal response value type, exposing just the surface FBRoute.m/FBResponseJSONPayload.m
+// call on it.
 
 @import Foundation;
 #import <WebDriverAgentLib/FBHTTPStatusCodes.h>

package/WebDriverAgentLib/Utilities/FBMjpegServer.h

@@ -19,14 +19,12 @@ NS_ASSUME_NONNULL_BEGIN
  */
 - (instancetype)init;
 
-#if TARGET_OS_WATCH
 /**
- The socket that owns this instance as its delegate. watchOS clients are bare nw_connection_t
- values with no write method of their own (unlike GCDAsyncSocket on other platforms), so frame
- writes are routed through -[FBTCPSocket writeData:toClient:]. Must be set before streaming starts.
+ The socket that owns this instance as its delegate. Clients are bare nw_connection_t values
+ with no write method of their own, so frame writes are routed through
+ -[FBTCPSocket writeData:toClient:]. Must be set before streaming starts.
  */
 @property (nonatomic, weak, nullable) FBTCPSocket *socket;
-#endif
 
 /**
  Stops screenshot broadcasting and prevents future scheduling.

package/WebDriverAgentLib/Utilities/FBMjpegServer.m

@@ -11,12 +11,8 @@
 #import <mach/mach_time.h>
 @import UniformTypeIdentifiers;
 
-#if TARGET_OS_WATCH
 // Textual import, not `@import Network;` - see the comment in FBTCPSocket.h.
 #import <Network/Network.h>
-#else
-#import "GCDAsyncSocket.h"
-#endif
 #import "FBConfiguration.h"
 #import "FBLogger.h"
 #import "FBScreenshot.h"
@@ -41,11 +37,7 @@ static NSUInteger FBNormalizedMjpegFramerate(NSUInteger framerate)
 @interface FBMjpegServer()
 
 @property (nonatomic, readonly) dispatch_queue_t backgroundQueue;
-#if TARGET_OS_WATCH
 @property (nonatomic, readonly) NSMutableArray<nw_connection_t> *listeningClients;
-#else
-@property (nonatomic, readonly) NSMutableArray<GCDAsyncSocket *> *listeningClients;
-#endif
 @property (nonatomic, readonly) FBImageProcessor *imageProcessor;
 @property (nonatomic, readonly) long long mainScreenID;
 @property (nonatomic, assign) NSUInteger consecutiveScreenshotFailures;
@@ -157,16 +149,9 @@ static NSUInteger FBNormalizedMjpegFramerate(NSUInteger framerate)
       return;
     }
     NSUInteger clientCount = self.listeningClients.count;
-#if TARGET_OS_WATCH
     for (nw_connection_t client in self.listeningClients) {
       [self.socket writeData:chunk toClient:client];
     }
-#else
-    for (GCDAsyncSocket *client in self.listeningClients) {
-      // Slow clients should fail/close instead of buffering indefinitely.
-      [client writeData:chunk withTimeout:FRAME_TIMEOUT tag:0];
-    }
-#endif
     self.sentFramesCount++;
     self.sentBytesCount += chunk.length * clientCount;
     NSUInteger framerate = FBNormalizedMjpegFramerate(FBConfiguration.sharedInstance.mjpegServerFramerate);
@@ -179,13 +164,10 @@ static NSUInteger FBNormalizedMjpegFramerate(NSUInteger framerate)
   }
 }
 
-#if TARGET_OS_WATCH
-
 - (void)didClientConnect:(nw_connection_t)newClient
 {
   [FBLogger log:@"Got screenshots broadcast client connection"];
-  // FBTCPSocket already schedules the receive that -client:didReceiveData: relies on below;
-  // unlike GCDAsyncSocket, there is nothing to arm here.
+  // FBTCPSocket already schedules the receive that -client:didReceiveData: relies on below.
 }
 
 - (void)client:(nw_connection_t)client didReceiveData:(NSData *)data
@@ -224,53 +206,6 @@ static NSUInteger FBNormalizedMjpegFramerate(NSUInteger framerate)
   }
 }
 
-#else
-
-- (void)didClientConnect:(GCDAsyncSocket *)newClient
-{
-  [FBLogger logFmt:@"Got screenshots broadcast client connection at %@:%d", newClient.connectedHost, newClient.connectedPort];
-  // Start broadcast only after there is any data from the client
-  [newClient readDataWithTimeout:-1 tag:0];
-}
-
-- (void)didClientSendData:(GCDAsyncSocket *)client
-{
-  @synchronized (self.listeningClients) {
-    if ([self.listeningClients containsObject:client]) {
-      return;
-    }
-  }
-
-  [FBLogger logFmt:@"Starting screenshots broadcast for the client at %@:%d", client.connectedHost, client.connectedPort];
-  NSString *streamHeader = [NSString stringWithFormat:@"HTTP/1.0 200 OK\r\nServer: %@\r\nConnection: close\r\nMax-Age: 0\r\nExpires: 0\r\nCache-Control: no-cache, private\r\nPragma: no-cache\r\nContent-Type: multipart/x-mixed-replace; boundary=--BoundaryString\r\n\r\n", SERVER_NAME];
-  [client writeData:(id)[streamHeader dataUsingEncoding:NSUTF8StringEncoding] withTimeout:-1 tag:0];
-  @synchronized (self.listeningClients) {
-    [self.listeningClients addObject:client];
-  }
-}
-
-- (void)didClientDisconnect:(GCDAsyncSocket *)client
-{
-  @synchronized (self.listeningClients) {
-    [self.listeningClients removeObject:client];
-  }
-  [FBLogger log:@"Disconnected a client from screenshots broadcast"];
-}
-
-- (void)stopStreaming
-{
-  self.isStreaming = NO;
-  @synchronized (self.listeningClients) {
-    NSArray<GCDAsyncSocket *> *clients = self.listeningClients.copy;
-    [self.listeningClients removeAllObjects];
-    for (GCDAsyncSocket *client in clients) {
-      [client disconnect];
-    }
-  }
-}
-
-#endif
-
 - (void)dealloc
 {
   [self stopStreaming];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "appium-webdriveragent",
-  "version": "16.6.0",
+  "version": "16.7.0",
   "description": "Package bundling WebDriverAgent",
   "keywords": [
     "Appium",