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

Migrate GandiProvider to new LiveDns api #639

Merged
merged 3 commits into from
Nov 30, 2023
Merged

Conversation

bhubert
Copy link
Contributor

@bhubert bhubert commented Nov 23, 2023

Gandi API has changed and migrate to new endpoints

@shibayan
Copy link
Owner

This appears to be a disruptive change to users using v4, has it been tested to be OK?

@bhubert
Copy link
Contributor Author

bhubert commented Nov 24, 2023

This appears to be a disruptive change to users using v4, has it been tested to be OK?

i tested my new version since i'm generating certificate for my company with my new fork and it's working great !

But for users on v4 i can't test it since my company is on v5. I'm just sure that the current version is not working on gandi v5 (the security header itself has changed from X-API-KEY to classic Authorization for example).

If you prefer i can add a option like UseLiveDNS in the Gandi Provider which will be at false by default and handle the two apis (v4 and v5) in the provider

@bhubert bhubert force-pushed the master branch 2 times, most recently from a6b2b5f to 4d6c965 Compare November 24, 2023 08:05
@bhubert
Copy link
Contributor Author

bhubert commented Nov 24, 2023

@shibayan I reworked the code to make it work with legacy api by default and allow new gandi users to use the new api

@shibayan
Copy link
Owner

Do not change the existing Gandi DNS implementation, but add the new implementation as GandiV5

@bhubert
Copy link
Contributor Author

bhubert commented Nov 24, 2023

Do not change the existing Gandi DNS implementation, but add the new implementation as GandiV5

The old one was already using "V5" in its path, its not really the version that changed, its more gandi that created new api endpoints (that started with v5).

So i changed as you asked by adding a new Provider but i called it GandiLiveDnsProvider and not GandiV5.

Hope it will suit you, thanks for all your fast feedbacks !

@shibayan shibayan self-requested a review November 25, 2023 05:48
@bhubert
Copy link
Contributor Author

bhubert commented Nov 28, 2023

@shibayan everything looks green, do you know when you will find time to package a new version ? i'd rather use an official version than my fork on production. thanks !

Copy link
Owner

@shibayan shibayan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@shibayan shibayan merged commit b035c8a into shibayan:master Nov 30, 2023
1 check passed
@bhubert
Copy link
Contributor Author

bhubert commented Nov 30, 2023

@shibayan thanks for the validation ! and thanks for all your hard work on this project !

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

Successfully merging this pull request may close these issues.

2 participants