Skip to content

Commit

Permalink
grpc-js: Don't include the port in :authority
Browse files Browse the repository at this point in the history
  • Loading branch information
murgatroid99 committed Apr 10, 2020
1 parent 9485062 commit f1c1daf
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions packages/grpc-js/src/resolver-dns.ts
Original file line number Diff line number Diff line change
Expand Up @@ -298,16 +298,10 @@ class DnsResolver implements Resolver {
IPV6_REGEX.exec(target) ||
IPV6_BRACKET_REGEX.exec(target);
if (ipMatch) {
if (ipMatch[2]) {
return ipMatch[1] + ':' + ipMatch[2];
}
return ipMatch[1];
}
const dnsMatch = DNS_REGEX.exec(target);
if (dnsMatch) {
if (dnsMatch[2]) {
return dnsMatch[1] + ':' + dnsMatch[2];
}
return dnsMatch[1];
}
throw new Error(`Failed to parse target ${target}`);
Expand Down

0 comments on commit f1c1daf

Please sign in to comment.