Skip to content
New issue

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

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

Already on GitHub? # to your account

OVH client expose endpoint #73

Closed
Azanul opened this issue Oct 9, 2023 · 3 comments · Fixed by #74
Closed

OVH client expose endpoint #73

Azanul opened this issue Oct 9, 2023 · 3 comments · Fixed by #74

Comments

@Azanul
Copy link

Azanul commented Oct 9, 2023

I need to access the endpoint of a created client. Without it I've to pass the endpoint through my functions separately.

This can be done in a backward compatible way by adding a method to get the endpoint.

@deathiop
Copy link
Contributor

deathiop commented Oct 9, 2023

Hello @Azanul,

May you please elaborate on your exact need? Maybe with a small example of what you fail to achieve with go-ovh in its current state?

@Azanul
Copy link
Author

Azanul commented Oct 9, 2023

Hi @deathiop,
I'm creating an integration to Komiser
I need endpoint

endpoint string
. I can achieve the same by passing endpoint separately along with the client but it'd be better if endpoint was exposed from the client itself.

@deathiop
Copy link
Contributor

deathiop commented Oct 9, 2023

You can upgrade to v1.4.3 which allow client.Endpoint() as a getter for you need.
See #74
Have a nice day!

@deathiop deathiop closed this as completed Oct 9, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants