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

Bug in ISPConfig DNS Challenge #3884

Open
lcx opened this issue Jul 19, 2024 · 2 comments
Open

Bug in ISPConfig DNS Challenge #3884

lcx opened this issue Jul 19, 2024 · 2 comments
Labels

Comments

@lcx
Copy link

lcx commented Jul 19, 2024

Describe the bug

Using ISPConfig as DNS Challenge for a wildcard Let's Encrypt certificate request fails with

API response with an error: Incorrect datetime value: '' for column `dbispconfig`.`dns_rr`.`stamp` at row 1 INSERT INTO `dns_rr` (`server_id`, `zone`, `name`, `type`, `data`, `ttl`, `active`, `stamp`, `serial`, `sys_userid`, `sys_groupid`, `sys_perm_user`, `sys_perm_group`, `sys_perm_other`) VALUES ('1', '2', '_acme-challenge', 'TXT', 'asdfg', '60', 'Y', '', '0', '1', '1', 'riud', 'riud', '')

The reason for this is that the stamp is a Timestamp field but no value is provided in the API request.

I have reported this with ISPConfig as well and also posted a workaround here

As far as I could debug the API call it sends the _acme-challenge + the key that is added in the text field which makes ISPconfig use the provided default value of '' that lead to the mysql Error.

Versions

Nginx Proxy Manager Version: v2.11.3
ISPConfig: v3.2.12

To Reproduce
Steps to reproduce the behavior:

  1. Go to SSL Certificates
  2. Click on Add SSL Certificate
  3. Select Let's encrypt
  4. Select Use DNS Challenge
  5. Pick ISPConfig from the list
  6. Provider ISP Config data
  7. click Save
  8. The mentioned Error appears.

How can I help?

If someone shows me where the ISPConfig thing is in NPM I could try to create a pull request since I have a ISPConfig at my disposition to test with.

@lcx lcx added the bug label Jul 19, 2024
@lcx
Copy link
Author

lcx commented Jul 19, 2024

ok Seems this is actually coming from certbot-dns-ispconfig and there is already an issue and is suppossedly already fixed but pip install get's the wrong (older) version. Guess this can be closed as I don't see why and how nginx-proxy-manager could fix that.

@rabesocke
Copy link

There is a new version of the certbot-dns-ispconfig plugin. Is there anything to do in this project to test and use this version?

https://pypi.org/project/certbot-dns-ispconfig/

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

No branches or pull requests

2 participants