We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
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
The http adaptor does this:
export function addBasicAuth(configuration = {}, headers) { const { username, password } = configuration; if (username && password) { Object.assign(headers, makeBasicAuthHeader(username, password)); } }
(probably other adaptors too)
This blows up on v1 if there's no credential, because v1 passesnull as the credential. And null is a value, it won't trigger the default.
null
While we're still supporting v1, we need to be careful about defaulting the configuration object across our adaptors.
configuration
we should really fix this in http because latest will be forever broken on v1, which isn't so good
latest
The text was updated successfully, but these errors were encountered:
mtuchi
No branches or pull requests
The http adaptor does this:
(probably other adaptors too)
This blows up on v1 if there's no credential, because v1 passes
null
as the credential. Andnull
is a value, it won't trigger the default.While we're still supporting v1, we need to be careful about defaulting the
configuration
object across our adaptors.we should really fix this in http because
latest
will be forever broken on v1, which isn't so goodThe text was updated successfully, but these errors were encountered: