Skip to content

feat: Add DNS challenge provider support for Aliyun ESA.#5639

Open
lampofaladdin wants to merge 1 commit into
NginxProxyManager:developfrom
lampofaladdin:develop
Open

feat: Add DNS challenge provider support for Aliyun ESA.#5639
lampofaladdin wants to merge 1 commit into
NginxProxyManager:developfrom
lampofaladdin:develop

Conversation

@lampofaladdin
Copy link
Copy Markdown

Why

Add DNS challenge provider support for Aliyun ESA.

Aliyun ESA uses a different Certbot DNS plugin from the existing Aliyun DNS provider. This PR adds a new provider entry for:

Credentials template:

dns_aliyun_esa_access_key_id = 12345678
dns_aliyun_esa_access_key_secret = 1234567890abcdef1234567890abcdef

This is a non-breaking addition and does not change the existing Aliyun provider.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • API changes
  • Performance improvement
  • Test addition or update

AI Usage

  • AI was used to write this
  • AI was used to review this

Snapshot

  • Certbot Successfully received certificate.
37c07441bf837e51ecc328b3dc0f6ec8
  • NPM select UI
84b3e88691bb008dec573b29aaf24af9
  • Aliyun ESA
a3835bd3-d0c6-4eb5-b7ba-4984f54a365a
  • NPM Success
fcec355aea3cd299edf5de06680e1f40

@nginxproxymanagerci
Copy link
Copy Markdown

Docker Image for build 1 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-5639

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

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.

1 participant