Removed Node transport from automatic detection

README.md

@@ -144,7 +144,14 @@ This library is tested against:
 
 ## Node.js Support
 
-`grpc-web-client` also [supports Node.js through a transport](ts/docs/transport.md#node-http-only-available-in-a-nodejs-environment) that uses the `http` and `https` packages. Usage does not vary from browser usage as transport is determined at runtime.
+`grpc-web-client` also [supports Node.js through a transport](ts/docs/transport.md#node-http-only-available-in-a-nodejs-environment) that uses the `http` and `https` packages. In order to ensure the library is tree-shakable, you must explicitly import and specify the Node transport like so:
+
+```
+import nodeHttpRequest from "grpc-web-client/dist/transports/nodeHttp";
+grpc.unary(BookService.QueryBooks, {
+  transport: nodeHttpRequest
+});
+```
 
 If you want to use `grpc-web-client` in a node.js environment with Typescript, you must include `dom` in the `"lib"` Array in your `tsconfig.json` otherwise `tsc` will be unable to find some type declarations to compile. Note that `dom` will be included automatically if you do not declare `lib` in your configration and your target is one of `es5` or `es6`. (See [Typescript compiler options](https://www.typescriptlang.org/docs/handbook/compiler-options.html)).
 

test/ts/src/testRpcCombinations.ts

@@ -3,6 +3,7 @@ import {
   corsHost
 } from "../../hosts-config";
 import {grpc} from "../../../ts/src/index";
+import nodeHttpRequest, {detectNodeHTTPSupport} from "../../../ts/src/transports/nodeHttp";
 
 type TestConfig = {
   testHostUrl: string,
@@ -54,7 +55,7 @@ export function runWithHttp1AndHttp2(cb: (config: TestConfig) => void) {
 
 export function runWithSupportedTransports(cb: (transport: grpc.TransportConstructor | undefined) => void) {
   const transports: {[key: string]: grpc.TransportConstructor | undefined} = {
-    "defaultTransport": undefined
+    "defaultTransport": detectNodeHTTPSupport() ? nodeHttpRequest : undefined
   };
 
   if (!process.env.DISABLE_WEBSOCKET_TESTS) {

ts/src/transports/Transport.ts

@@ -2,7 +2,6 @@ import {Metadata} from "../metadata";
 import fetchRequest, {detectFetchSupport} from "./fetch";
 import xhrRequest, {detectXHRSupport} from "./xhr";
 import mozXhrRequest, {detectMozXHRSupport} from "./mozXhr";
-import httpNodeRequest, {detectNodeHTTPSupport} from "./nodeHttp";
 import {MethodDefinition} from "../service";
 import {ProtobufMessage} from "../message";
 import websocketRequest from "./websocket";
@@ -54,11 +53,8 @@ function detectTransport(): TransportConstructor {
     return xhrRequest;
   }
 
-  if (detectNodeHTTPSupport()) {
-    return httpNodeRequest;
-  }
-
-  throw new Error("No suitable transport found for gRPC-Web");
+  throw new Error("No suitable transport found for gRPC-Web.  If running in " +
+    "Node, import and set the `transport` option to nodeHttpRequest");
 }
 
 export function WebsocketTransportFactory(transportOptions: TransportOptions): Transport | Error {