Skip to content

feat: deploy agent with authentication#302

Merged
warm-wm merged 7 commits intovolcengine:mainfrom
zhxie:deploy-oauth
Nov 17, 2025
Merged

feat: deploy agent with authentication#302
warm-wm merged 7 commits intovolcengine:mainfrom
zhxie:deploy-oauth

Conversation

@zhxie
Copy link
Contributor

@zhxie zhxie commented Nov 11, 2025

Signed-off-by: Xie Zhihao zhihao.xie@bytedance.com

The patch adds several authentications for web application and agents.

  1. Add API key authentication.
  2. Add OAuth2 SSO for web application.
  3. Add JWT authentication for agents.

Signed-off-by: Xie Zhihao <zhihao.xie@bytedance.com>
Signed-off-by: Xie Zhihao <zhihao.xie@bytedance.com>
Signed-off-by: Xie Zhihao <zhihao.xie@bytedance.com>
Signed-off-by: Xie Zhihao <zhihao.xie@bytedance.com>
Signed-off-by: Xie Zhihao <zhihao.xie@bytedance.com>
Signed-off-by: Xie Zhihao <zhihao.xie@bytedance.com>
| `--veapig-instance-name` | 字符串 | 火山引擎 APIG 实例名称 |
| `--veapig-service-name` | 字符串 | 火山引擎 APIG 服务名称 |
| `--veapig-upstream-name` | 字符串 | 火山引擎 APIG Upstream 名称 |
| `--vefaas-app-name` | 字符串 | 火山引擎函数服务应用名称 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

why we need the app name for newly deployed app?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This parameter existed already. I just changed its description.

Copy link
Collaborator

@cuericlee cuericlee left a comment

Choose a reason for hiding this comment

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

/lgtm

@cuericlee cuericlee marked this pull request as ready for review November 14, 2025 10:51
Copy link
Collaborator

@cuericlee cuericlee left a comment

Choose a reason for hiding this comment

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

/lgtm

@warm-wm warm-wm merged commit 112a19b into volcengine:main Nov 17, 2025
5 checks passed
@zhxie zhxie deleted the deploy-oauth branch November 17, 2025 03:06
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.

4 participants