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

Contribute dns_zoneedit.sh to acme.sh official repo #13

Open
lukavia opened this issue Dec 5, 2024 · 8 comments
Open

Contribute dns_zoneedit.sh to acme.sh official repo #13

lukavia opened this issue Dec 5, 2024 · 8 comments

Comments

@lukavia
Copy link

lukavia commented Dec 5, 2024

Have you considered contributing your code to the official acme.sh project?
That way it would be eventually available in pfsense and opnsense without the need to patch after each update.

If you don't have the time do you mind if I try to make pull request to them following their guides?

Thank you for this wonderful work

@sbn-purchark
Copy link
Contributor

sbn-purchark commented Dec 5, 2024

I sure haven't been able to find the time. I basically stopped working on it once it was working good. (2 years ago)

It does not fully meet code conformity or integration requirements, but I don't know how picky they will be about that. I think it was mostly variable names or formatting related and I noted that I got it mostly there.

(I contributed the rewrite)

And I guess ZoneEdit never did follow through with their API changes.

@lukavia
Copy link
Author

lukavia commented Dec 6, 2024

We'll see: acmesh-official/acme.sh#6136

@blueslow
Copy link
Owner

blueslow commented Dec 6, 2024

In the beginning I read the requirements of the acme.sh project but at that time the zonedit api was not complete, so I dropped the idea. When sbn-purchark provided improvement the zoneedit api was enhanced. At that time and since then I didn't had the time to continue. Lukavia you are welcome to give it a shoot. :-)

@lukavia
Copy link
Author

lukavia commented Dec 9, 2024

OK. So the pull request is merged in acmesh-official:dev
I don't know how much time it would take to get in master and in release.

Do you want to keep this issue open and notify you or should we close it?

@blueslow
Copy link
Owner

Thanks Lukavia. I will leave it open until we know the results. :-)

@lukavia
Copy link
Author

lukavia commented Dec 11, 2024

It's already in master, so it should be included in the next release I would think.

@stokito
Copy link

stokito commented Dec 28, 2024

It also would be nice if you can subscribe to the support topic github.com/acmesh-official/acme.sh/issues/6135

@blueslow
Copy link
Owner

It also would be nice if you can subscribe to the support topic github.com/acmesh-official/acme.sh/issues/6135

I think I have successfully subscribed to the issue.

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

4 participants