-
Notifications
You must be signed in to change notification settings - Fork 3.2k
feat: support certificate synchronization #13153
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -183,9 +183,9 @@ var WebUrlMap = map[string]struct{}{ | |
| "/xpack/node/dashboard": {}, | ||
| "/xpack/node": {}, | ||
| "/xpack/simple-node": {}, | ||
| "/xpack/sync": {}, | ||
| "/xpack/sync/file": {}, | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The menu data still points the Resource Sync entry at Useful? React with 👍 / 👎. |
||
| "/xpack/sync/image": {}, | ||
| "/xpack/sync/cert": {}, | ||
| "/xpack/sync/ssl": {}, | ||
| "/xpack/sync/app": {}, | ||
| "/xpack/app": {}, | ||
| "/xpack/app-upgrade": {}, | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When
/apps/installed/searchbuilds its normal listing throughhandleInstalled, this now serializesinstalled.Envfor every returned app. That field is populated from install parameters and is used elsewhere for secrets such asPANEL_DB_ROOT_PASSWORDandREPLICATION_PASSWORD, so app/website listing calls that previously returned metadata now bulk-expose all installed-app environment secrets; keep this data behind the per-install/detail or sync-specific path instead of adding it to the generic DTO.Useful? React with 👍 / 👎.