If jwksUri contains a port number, DNS resolution fails

[gyszalai]

Describe the problem

JwksClient uses hostname:port for hostname resolution so if the jwksUri contains a port number, hostname resolution fails.
The cause of the problem if that the request wrapper uses the host property from the parsed URL instead of the hostname property.

What was the expected behavior?

Hostname resolution should work even if the jwksUri contains a port number

Reproduction

import JwksClient from 'jwks-rsa'

const jwksClient = JwksClient({
  jwksUri: 'http://myhost:8888/jwks'
})

async function getKey(keyId: string) => {
  const key = await jwksClient.getSigningKey(keyId)
  return { cert: key.getPublicKey() }
}

const key = await getKey('1)

Environment

• Version of this library used: 2.0.1
• Which framework are you using, if applicable: -
• Other modules/plugins/libraries that might be involved: -
• Any other relevant information you think would be useful: -

[carboneater]

Looks like we have a proposed solution in #233

[gyszalai]

Sorry, my fault. Didn't check PRs before filing the issue.

[jonjrodriguez]

Having the same issue when trying to use localstack.

[davidpatrick]

Fixed in #233