Skip to content

feat: support gcp secret manager#11436

Merged
bzp2010 merged 28 commits into
apache:masterfrom
HuanXin-Chen:feat-gcp-secret
Sep 22, 2024
Merged

feat: support gcp secret manager#11436
bzp2010 merged 28 commits into
apache:masterfrom
HuanXin-Chen:feat-gcp-secret

Conversation

@HuanXin-Chen

Copy link
Copy Markdown
Contributor

Description

Many enterprises are utilizing cloud services from AWS and GCP, relying on the secret manager provided by these platforms to handle sensitive information. Integrating Apache APISIX with these secret managers can streamline the process of using sensitive information within APISIX, enabling users to manage and utilize cloud-stored sensitive data more conveniently, thus enhancing the overall security and usability of the system.

This PR has completed the support for GCP. It added the gcp.lua file to the original secret module, allowing users to store their secret information on GCP using the same reference method as before.

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)

Comment thread apisix/secret/gcp.lua Outdated
@dosubot dosubot Bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Aug 9, 2024
@HuanXin-Chen
HuanXin-Chen requested a review from soulbird August 25, 2024 03:49
@bzp2010
bzp2010 self-requested a review September 4, 2024 07:40
Comment thread apisix/utils/google-cloud-oauth.lua Outdated
Comment thread apisix/utils/google-cloud-oauth.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread t/secret/conf/success.json
@bzp2010
bzp2010 requested a review from kayx23 September 4, 2024 08:02
@bzp2010

bzp2010 commented Sep 4, 2024

Copy link
Copy Markdown
Contributor

Hi @kayx23

Please take a look at the documentation when you are free to see if there are any problems. 🫡

Comment thread apisix/utils/google-cloud-oauth.lua Outdated
Comment thread apisix/utils/google-cloud-oauth.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/utils/google-cloud-oauth.lua Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua
Comment thread apisix/secret/gcp.lua
Comment thread t/secret/gcp.t Outdated
Comment thread docs/en/latest/terminology/secret.md
Comment thread t/secret/gcp.t Outdated
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/utils/google-cloud-oauth.lua
@HuanXin-Chen

Copy link
Copy Markdown
Contributor Author

@bzp2010 @kayx23 @membphis @soulbird I have fixed some issues in the code. Please review it again when you have time. Thank you very much. 🙏

@bzp2010

bzp2010 commented Sep 12, 2024

Copy link
Copy Markdown
Contributor

@HuanXin-Chen There were some failures in CI involving traffic-split3.t, no worries, just merge the master branch into your dev branch feat-gcp-secret, we've fixed that.

Comment thread apisix/secret/gcp.lua
Comment thread apisix/secret/gcp.lua Outdated
Comment thread apisix/secret/gcp.lua
@HuanXin-Chen

Copy link
Copy Markdown
Contributor Author

@bzp2010 @membphis @soulbird @kayx23 @shreemaan-abhishek
I think there's no problem in my pr. Some failures in CI are not related to my code, no worries, just retry may solve it.
Please review it when you're free. Only after this branch is merged can I proceed with the next step of work: #11576. Thank you!💗

membphis
membphis previously approved these changes Sep 18, 2024

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, I mainly check the code style, doc and test case

Comment thread t/secret/gcp.t Outdated

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thx for your nice PR

@bzp2010
bzp2010 merged commit 2fcfbd8 into apache:master Sep 22, 2024
@bzp2010

bzp2010 commented Sep 22, 2024

Copy link
Copy Markdown
Contributor

Great job 🫡

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants