Skip to content

grpc-js: Use a more structured representation of URIs internally #1364

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
murgatroid99 merged 5 commits into from
Apr 20, 2020
Merged

grpc-js: Use a more structured representation of URIs internally #1364

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
13cc016
grpc-js: Use a more structured representation of URIs internally
murgatroid99 Apr 16, 2020
1cd0ef1
Fix target name in "other" resolver test
murgatroid99 Apr 16, 2020
dbe98ad
Fix "other" resovler registration code
murgatroid99 Apr 16, 2020
5aa7817
Merge branch 'master' into grpc-js_uri_parsing
murgatroid99 Apr 20, 2020
98e4626
Fix merge error with proxy fixes
murgatroid99 Apr 20, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix merge error with proxy fixes
  • Loading branch information
@murgatroid99
murgatroid99 committed Apr 20, 2020
commit 98e46260ef0e77a1052a2c51ca468de60e7eb4b1
2 changes: 1 addition & 1 deletion packages/grpc-js/src/http_proxy.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ export function getProxiedConnection(
* See https://github.com/grpc/grpc-node/pull/1369 for more info. */
const cts = tls.connect({
...connectionOptions,
host: getDefaultAuthority(realTarget),
host: getDefaultAuthority(parsedTarget),
socket: socket,
}, () => {
resolve({ socket: cts, realTarget: parsedTarget });
Expand Down