Skip to content

feat: support store route's cert in secrets manager#9247

Merged
juststillthinking merged 15 commits into
apache:masterfrom
soulbird:feat/cert-secret
Apr 12, 2023
Merged

feat: support store route's cert in secrets manager#9247
juststillthinking merged 15 commits into
apache:masterfrom
soulbird:feat/cert-secret

Conversation

@soulbird

@soulbird soulbird commented Apr 6, 2023

Copy link
Copy Markdown
Contributor

Description

As part of #8319

Support for storing certificates on routers through Secret Manager.
When using this function to configure the certificate, we only verify the validity of the certificate when the certificate is used on the DP side. This is done to avoid the problem that the CP side may not be able to access the backend Secret Manager Service.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@soulbird
soulbird marked this pull request as ready for review April 6, 2023 06:18
kingluo
kingluo previously approved these changes Apr 6, 2023
Comment thread apisix/schema_def.lua Outdated
Comment thread apisix/ssl.lua
Comment thread t/router/radixtree-sni2.t Outdated
Comment thread docs/en/latest/admin-api.md
Comment thread t/router/radixtree-sni2.t Outdated
Comment thread apisix/ssl.lua Outdated
Comment thread apisix/ssl.lua Outdated
Comment thread apisix/ssl/router/radixtree_sni.lua
Comment thread apisix/ssl.lua Outdated
@juststillthinking

Copy link
Copy Markdown
Contributor

please make ci pass

Comment thread apisix/ssl.lua Outdated
@juststillthinking
juststillthinking merged commit ab0a867 into apache:master Apr 12, 2023
shreemaan-abhishek pushed a commit to shreemaan-abhishek/apisix that referenced this pull request Jan 2, 2026
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.

3 participants