Skip to content
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.

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

AuthenticatedClient doesn't respect the ClientOpts.HTTPClient setting #192

Open
kayrus opened this issue Jun 23, 2023 · 3 comments
Open
Assignees

Comments

@kayrus
Copy link
Contributor

kayrus commented Jun 23, 2023

ao, err := AuthOptions(opts)
call just ignores the HTTPClient member.

@pierreprinetti
Copy link
Contributor

Hi @kayrus, thanks for reporting.

The gophercloud.AuthOptions object that func AuthOptions returns does not contain any HTTPClient, and apparently AuthOptions itself doesn't do any HTTP call. I believe that the HTTPClient contained in ClientOpts is intended for use in func NewServiceClient.

Please provide a minimal reproducer, and be specific about the behaviour you expected, so that I can better understand if we're missing something in code or in the docs.

@kayrus
Copy link
Contributor Author

kayrus commented Jun 23, 2023

See a linked issue. I think velero plugin author expected that providing ClientOpts.HTTPClient inherits it for the resulting provider client, but apparently it doesn't. I think it's a confusion, which can be fixed in utils.

@pierreprinetti
Copy link
Contributor

OK, understood. Please feel free to propose a solution in words or in code; I'll be happy to review.
If instead you'd like me to investigate the issue, I'll need a minimal reproducer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants