Commit Graph

3144 Commits

Author SHA1 Message Date
Michael Lumish afd33a71e6
Merge pull request #1611 from murgatroid99/proxy_port_80_fix_backport
Backport of "Set the default port of 80 explicitly in http_proxy" to 1.1.x
2020-10-28 13:32:17 -07:00
Michael Lumish da8088b955 Bump version to 1.1.8 2020-10-28 09:56:14 -07:00
Michael Lumish eb95fa618d Set the default port of 80 explicitly in http_proxy 2020-10-28 09:55:05 -07:00
Michael Lumish b51afd3852
Merge pull request #1580 from murgatroid99/grpc-js_event_loop_liveness
grpc-js: Add a timer that holds the event loop open while there are pending calls
2020-09-17 14:38:34 -07:00
Michael Lumish a1600812d1 Move timer initialization to the beginning of channel construction 2020-09-17 13:18:11 -07:00
Michael Lumish 6a32c00ed2 Bump grpc-js to 1.1.7 2020-09-17 12:15:47 -07:00
Michael Lumish 2791dbbb23 grpc-js: Add a timer that holds the event loop open while there are pending calls 2020-09-17 11:25:56 -07:00
Michael Lumish 2445875e08
Merge pull request #1562 from murgatroid99/grpc-js_1.1.6
Bump grpc-js to 1.1.6
2020-09-08 11:59:20 -07:00
Michael Lumish d8e00689d0 Bump grpc-js to 1.1.6 2020-09-03 14:32:30 -07:00
Michael Lumish f2d3b4d1ed
Merge pull request #1560 from grpc/revert-1556-master
Revert "fix(grpc-js): Add support for impl type to server.addService"
2020-09-02 16:04:42 -07:00
Michael Lumish 1fc284f59d
Revert "fix(grpc-js): Add support for impl type to server.addService" 2020-09-02 15:17:57 -07:00
Michael Lumish e21f3374e8
Merge pull request #1556 from slavovojacek/master
fix(grpc-js): Add support for impl type to server.addService
2020-09-02 13:39:20 -07:00
Michael Lumish 1178d553e0
Merge pull request #1557 from mad-it/master
fix(grpc-js): preserve original error code when the handler of a readable stream throws an error
2020-09-02 13:38:20 -07:00
Michael Lumish e0b19e5fd3
Merge pull request #1559 from grissius/fix/1558-broken-client-option-types
Fix ClientOptions types
2020-09-02 13:37:37 -07:00
Slavo Vojacek fba1ee0cc3 fix(grpc): Fix typings 2020-09-02 20:32:31 +01:00
Slavo Vojacek 0d1d5a12fa chore(Typings): Update types 2020-09-02 16:05:10 +01:00
Jaroslav Šmolík b2d89820a3 Fix ClientOptions types
Remove index signature from ChannelOptions to fix intersection error
described in #1558 which causes issues on using ClientOptions direct
fields with TypeScript.

Removing of index signature required few minor changes:
 - adding few constant types that were used throughout the app
 - using `as const` assertion in xds-client
 - using not-so-great type cast in channelOptionsEqual

Alternative solution would be removing the index signature from
ChannelOptions explicitly in ClientOptions definition, which is not
trivial and probably calls for a generic type helper.

See: https://github.com/grpc/grpc-node/issues/1558
Fixes: #1558
2020-09-01 21:47:52 +02:00
Algin Maduro aaee068a69 fix: add addition check if the provided code is valid gRPC code 2020-09-01 13:28:23 +02:00
Algin Maduro 5e42be1b34 fix: preserve original error code if present 2020-09-01 12:59:07 +02:00
Slavo Vojacek 38e988ea03 fix(grpc-js): Add support for impl type to server.addService 2020-08-31 20:34:14 +01:00
Michael Lumish 4b9da2ef7b
Merge pull request #1555 from wkchee/patch-1
Fixed connectivity to Google PubSub over proxy
2020-08-31 12:03:17 -07:00
WK 158d0dd99f
Update subchannel.ts 2020-09-01 01:40:14 +08:00
WK 7fc0035f7f
Update subchannel.ts 2020-09-01 01:30:35 +08:00
WK 148b273f19
Update http_proxy.ts 2020-09-01 01:28:53 +08:00
WK 6a99983ed1
Undo changes. 2020-08-30 14:56:23 +08:00
WK 08350ec0ef
Update subchannel.ts 2020-08-30 14:52:20 +08:00
WK 6f3db6f4d8
Update http_proxy.ts 2020-08-30 14:49:54 +08:00
WK c536178c67
Fixed connectivity to Google PubSub over proxy
Latest version has caused @google-cloud/pubsub fail to connect over a proxy connection.


Snapshot of error:

2020-08-29T10:52:45.340Z | proxy | Successfully connected to pubsub.googleapis.c
om:443 through proxy 172.16.52.252:443
2020-08-29T10:52:45.370Z | subchannel | 172.16.52.252:443 CONNECTING -> TRANSIEN
T_FAILURE
2020-08-29T10:52:45.372Z | pick_first | CONNECTING -> TRANSIENT_FAILURE
2020-08-29T10:52:45.373Z | resolving_load_balancer | dns:172.16.52.252:443 CONNE
CTING -> TRANSIENT_FAILURE
2020-08-29T10:52:45.375Z | channel | Pick result: TRANSIENT_FAILURE subchannel:
undefined status: 14 No connection established
2020-08-29T10:52:45.377Z | call_stream | [11] cancelWithStatus code: 14 details:
 "No connection established"
2020-08-29T10:52:45.379Z | call_stream | [11] ended with status: code=14 details
="No connection established"
2020-08-29T10:52:45.381Z | connectivity_state | dns:172.16.52.252:443 CONNECTING
 -> TRANSIENT_FAILURE


Before proposed fix:

    static getDefaultAuthority(target) {
        return target.path;  // this returns "pubsub.googleapis.com:443"
    }

After proposed fix:

    static getDefaultAuthority(target) {
        const hostPort = uri_parser_1.splitHostPort(target.path);  // target.path is "pubsub.googleapis.com:443"
        if (hostPort !== null) {
            return hostPort.host; // this returns "pubsub.googleapis.com"
        }
        else {
            throw new Error(`Failed to parse target ${uri_parser_1.uriToString(target)}`);
        }
    }
2020-08-29 19:05:42 +08:00
Michael Lumish 2111c0ffa9
Merge pull request #1553 from ably-forks/watchconnectivitystate-nodeadline
Channel#watchConnectivityState: handle infinite deadlines correctly
2020-08-24 10:47:11 -07:00
Simon Woolf 141dfeb790 Channel#watchConnectivityState: handle infinite deadlines correctly
Per https://grpc.github.io/grpc/node/grpc.html#~Deadline:
"If it is a finite number, it is treated as a number of milliseconds
since the Unix Epoch. If it is Infinity, the deadline will never be
reached. If it is -Infinity, the deadline has already passed."
2020-08-21 18:18:30 +01:00
Michael Lumish 69b197b75b
Merge pull request #1551 from murgatroid99/grpc-js_dependency_fix
grpc-js: Move a couple of dev dependencies to prod
2020-08-20 09:45:05 -07:00
Michael Lumish 5abb47390f grpc-js: Move a couple of dev dependencies to prod 2020-08-20 09:24:57 -07:00
Michael Lumish 7efc5358e3
Merge pull request #1524 from murgatroid99/grpc-js_cancelled_event
grpc-js: Clean up call even if status throws an error
2020-08-19 10:09:53 -07:00
Michael Lumish c0b80a3593
Merge pull request #1548 from richardpringle/master
Prevent mutation of default headers
2020-08-19 09:55:09 -07:00
Michael Lumish 68b0609feb
Merge pull request #1525 from murgatroid99/grpc-js_server_stream_end_metadata
grpc-js: Add end(md?: Metadata) method to streaming server calls
2020-08-19 09:29:52 -07:00
Michael Lumish 4ba1639ed6
Merge pull request #1526 from murgatroid99/grpc-js_get_peer
grpc-js: Implement getPeer on the client and server
2020-08-19 09:27:56 -07:00
Michael Lumish b771b994a8
Merge pull request #1546 from murgatroid99/grpc-js_types_node_dep
grpc-js: Move @types/node to a production dependency
2020-08-19 09:27:36 -07:00
Richard Pringle 917b4fca77
Prevent mutation of default headers 2020-08-19 10:24:20 -04:00
Michael Lumish 7493052672 grpc-js: Move @types/node to a production dependency 2020-08-18 15:13:12 -07:00
Michael Lumish a2f2f448b3
Merge pull request #1541 from tlhunter/patch-1
Add link to grpc docs in @grpc/grpc-js README
2020-08-17 10:09:29 -07:00
Thomas Hunter II 1583786478
Add link to grpc docs in @grpc/grpc-js README
- Adds a link to `grpc` documentation
- Addresses some of the concerns in #1540
2020-08-16 13:50:23 -07:00
Michael Lumish e37e4f26c6
Merge pull request #1534 from mavaa/patch-1
Fix incorrectly named grpc-tools flag
2020-08-12 09:46:33 -07:00
Martin f0ed1aba14
Fix incorrectly named grpc-tools flag
Was  "--generate_package_definitions" (with an s) but should be "generate_package_definition" as in the documentation here: https://github.com/grpc/grpc-node/tree/master/packages/grpc-tools
2020-08-12 16:51:42 +02:00
Michael Lumish 1687e3de07
Merge pull request #1528 from murgatroid99/grpc-js_xds_client_refactor
grpc-js: Refactor the XdsClient class
2020-08-07 10:05:28 -07:00
Michael Lumish 6c0012499a Implement onResourceDoesNotExist notifications 2020-08-06 13:30:24 -07:00
Michael Lumish 5dcce9ebf1 grpc-js: XdsClient: separate ADS stream handling by message type 2020-08-06 13:08:59 -07:00
Michael Lumish 2388e31f8d
Merge pull request #1522 from murgatroid99/grpc-js_xds_resolver
grpc-js: Add XdsResolver and corresponding XdsClient behavior
2020-08-06 10:00:07 -07:00
Michael Lumish 76bb17091b Stop processing RDS requests after finding the matching one 2020-08-05 12:57:54 -07:00
Michael Lumish 0b146c8b07 Address PR comments 2020-08-05 12:38:43 -07:00
Michael Lumish 33a4c85f89 grpc-js: Implement getPeer on the client and server 2020-08-04 13:04:59 -07:00