Skip to content
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

Move secrets and runners settings to actions settings #24200

Merged
merged 55 commits into from
Apr 28, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
55 commits
Select commit Hold shift + click to select a range
1b1e52c
move secrets to actions(repo)
HesterG Apr 19, 2023
f5737f7
update secretes and runners for org
HesterG Apr 19, 2023
118f1e1
modify admin
HesterG Apr 19, 2023
22f9c76
modify search
HesterG Apr 19, 2023
d20aad6
modify for user level
HesterG Apr 19, 2023
aa38122
revert comment
HesterG Apr 19, 2023
02cc857
hide user level action if not enabled
HesterG Apr 19, 2023
586b380
secrets management locale
HesterG Apr 19, 2023
17e77c6
refactor names
HesterG Apr 19, 2023
f3e2fbe
remove unused tmpl and comments
HesterG Apr 19, 2023
8a3af8b
fix some titles and use PageIsOrgSettings
HesterG Apr 19, 2023
177cfa4
fix lint and use string
HesterG Apr 19, 2023
40596aa
try adjust error position and pass ResetRegistrationTokenLink
HesterG Apr 19, 2023
e7d41b3
use some format of variable and delete runners/base
HesterG Apr 19, 2023
aa6aefb
change classname and css
HesterG Apr 19, 2023
7414385
rename
HesterG Apr 19, 2023
4674fc5
change ctx data passed to shared templates and add redirext
HesterG Apr 20, 2023
6d5cdd4
fix backend check
HesterG Apr 20, 2023
99781f2
change alert position
HesterG Apr 20, 2023
88306c0
rename function
HesterG Apr 20, 2023
c119639
use capital and move redirect to function
HesterG Apr 20, 2023
4903d7c
rename admin/settings_actions.go and improve org/setting_actions
HesterG Apr 20, 2023
d4acdc8
put settings in one folder
HesterG Apr 20, 2023
e849999
rename
HesterG Apr 20, 2023
311e953
fix lint
HesterG Apr 21, 2023
0e79e6e
make repo settings runners/secrets submenus
HesterG Apr 23, 2023
42f26c5
fix path and lint
HesterG Apr 24, 2023
867b373
make runner/secrets submenu on user and admin
HesterG Apr 24, 2023
0a68dda
change actions classname
HesterG Apr 24, 2023
d63330e
fix path
HesterG Apr 24, 2023
2019081
remove unnecessary change
HesterG Apr 24, 2023
b09c289
rename redirect function
HesterG Apr 24, 2023
a75ae88
rename file
HesterG Apr 24, 2023
244e568
separate routers for runners and secrets
HesterG Apr 25, 2023
a176331
adjust margin-left for vertical sub menu
HesterG Apr 25, 2023
f0788c6
use helper.css
HesterG Apr 25, 2023
fbd07c1
rename files
HesterG Apr 25, 2023
e0b4ec3
remove some changes
HesterG Apr 25, 2023
fbeb9ab
use variables for colors
HesterG Apr 25, 2023
38c8818
use text-indent instead of margin
HesterG Apr 25, 2023
bf43048
use the same rule
HesterG Apr 25, 2023
3d1def2
add custom class
HesterG Apr 27, 2023
def9495
Merge branch 'main' into move-secrets-to-actions
HesterG Apr 27, 2023
3d57d12
Merge branch 'main' into move-secrets-to-actions
HesterG Apr 27, 2023
fa112e5
use common handlers
HesterG Apr 27, 2023
ad374cd
add TODO
HesterG Apr 27, 2023
aca8a06
Merge branch 'main' into move-secrets-to-actions
HesterG Apr 27, 2023
42905d3
change conditions
HesterG Apr 27, 2023
bbb2c33
Merge branch 'main' into move-secrets-to-actions
HesterG Apr 27, 2023
4cd0166
simplify ctx data
wxiaoguang Apr 27, 2023
961d7bc
remove unnecessary PageIsAdmin
wxiaoguang Apr 27, 2023
ddd0661
remove g-settings-submenu
wxiaoguang Apr 27, 2023
d0d2e46
sub menu of vertical menu
wxiaoguang Apr 27, 2023
5d0644c
Merge branch 'main' into move-secrets-to-actions
GiteaBot Apr 27, 2023
1cf73f1
Merge branch 'main' into move-secrets-to-actions
GiteaBot Apr 27, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions options/locale/locale_en-US.ini
Original file line number Diff line number Diff line change
Expand Up @@ -3357,6 +3357,7 @@ deletion = Remove secret
deletion.description = Removing a secret is permanent and cannot be undone. Continue?
deletion.success = The secret has been removed.
deletion.failed = Failed to remove secret.
management = Secrets Management

[actions]
actions = Actions
Expand Down
5 changes: 0 additions & 5 deletions routers/web/admin/admin.go
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,6 @@ func updateSystemStatus() {
// Dashboard show admin panel dashboard
func Dashboard(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.dashboard")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminDashboard"] = true
ctx.Data["Stats"] = activities_model.GetStatistic()
ctx.Data["NeedUpdate"] = updatechecker.GetNeedUpdate()
Expand All @@ -129,7 +128,6 @@ func Dashboard(ctx *context.Context) {
func DashboardPost(ctx *context.Context) {
form := web.GetForm(ctx).(*forms.AdminDashboardForm)
ctx.Data["Title"] = ctx.Tr("admin.dashboard")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminDashboard"] = true
ctx.Data["Stats"] = activities_model.GetStatistic()
updateSystemStatus()
Expand All @@ -155,7 +153,6 @@ func DashboardPost(ctx *context.Context) {
// Monitor show admin monitor page
func Monitor(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.monitor")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminMonitor"] = true
ctx.Data["Processes"], ctx.Data["ProcessCount"] = process.GetManager().Processes(false, true)
ctx.Data["Entries"] = cron.ListTasks()
Expand All @@ -167,7 +164,6 @@ func Monitor(ctx *context.Context) {
// GoroutineStacktrace show admin monitor goroutines page
func GoroutineStacktrace(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.monitor")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminMonitor"] = true

processStacks, processCount, goroutineCount, err := process.GetManager().ProcessStacktraces(false, false)
Expand Down Expand Up @@ -202,7 +198,6 @@ func Queue(ctx *context.Context) {
return
}
ctx.Data["Title"] = ctx.Tr("admin.monitor.queue", mq.Name)
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminMonitor"] = true
ctx.Data["Queue"] = mq
ctx.HTML(http.StatusOK, tplQueue)
Expand Down
5 changes: 0 additions & 5 deletions routers/web/admin/applications.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ func newOAuth2CommonHandlers() *user_setting.OAuth2CommonHandlers {
// Applications render org applications page (for org, at the moment, there are only OAuth2 applications)
func Applications(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("settings.applications")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminApplications"] = true

apps, err := auth.GetOAuth2ApplicationsByUserID(ctx, 0)
Expand All @@ -47,7 +46,6 @@ func Applications(ctx *context.Context) {
// ApplicationsPost response for adding an oauth2 application
func ApplicationsPost(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("settings.applications")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminApplications"] = true

oa := newOAuth2CommonHandlers()
Expand All @@ -56,7 +54,6 @@ func ApplicationsPost(ctx *context.Context) {

// EditApplication displays the given application
func EditApplication(ctx *context.Context) {
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminApplications"] = true

oa := newOAuth2CommonHandlers()
Expand All @@ -66,7 +63,6 @@ func EditApplication(ctx *context.Context) {
// EditApplicationPost response for editing oauth2 application
func EditApplicationPost(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("settings.applications")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminApplications"] = true

oa := newOAuth2CommonHandlers()
Expand All @@ -76,7 +72,6 @@ func EditApplicationPost(ctx *context.Context) {
// ApplicationsRegenerateSecret handles the post request for regenerating the secret
func ApplicationsRegenerateSecret(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("settings")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminApplications"] = true

oa := newOAuth2CommonHandlers()
Expand Down
5 changes: 0 additions & 5 deletions routers/web/admin/auths.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ var (
// Authentications show authentication config page
func Authentications(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.authentication")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminAuthentications"] = true

var err error
Expand Down Expand Up @@ -89,7 +88,6 @@ var (
// NewAuthSource render adding a new auth source page
func NewAuthSource(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.auths.new")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminAuthentications"] = true

ctx.Data["type"] = auth.LDAP.Int()
Expand Down Expand Up @@ -237,7 +235,6 @@ func parseSSPIConfig(ctx *context.Context, form forms.AuthenticationForm) (*sspi
func NewAuthSourcePost(ctx *context.Context) {
form := *web.GetForm(ctx).(*forms.AuthenticationForm)
ctx.Data["Title"] = ctx.Tr("admin.auths.new")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminAuthentications"] = true

ctx.Data["CurrentTypeName"] = auth.Type(form.Type).String()
Expand Down Expand Up @@ -333,7 +330,6 @@ func NewAuthSourcePost(ctx *context.Context) {
// EditAuthSource render editing auth source page
func EditAuthSource(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.auths.edit")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminAuthentications"] = true

ctx.Data["SecurityProtocols"] = securityProtocols
Expand Down Expand Up @@ -369,7 +365,6 @@ func EditAuthSource(ctx *context.Context) {
func EditAuthSourcePost(ctx *context.Context) {
form := *web.GetForm(ctx).(*forms.AuthenticationForm)
ctx.Data["Title"] = ctx.Tr("admin.auths.edit")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminAuthentications"] = true

ctx.Data["SMTPAuths"] = smtp.Authenticators
Expand Down
1 change: 0 additions & 1 deletion routers/web/admin/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,6 @@ func shadowPassword(provider, cfgItem string) string {
// Config show admin config page
func Config(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.config")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminConfig"] = true

systemSettings, err := system_model.GetAllSettings(ctx)
Expand Down
1 change: 0 additions & 1 deletion routers/web/admin/emails.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ const (
// Emails show all emails
func Emails(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.emails")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminEmails"] = true

opts := &user_model.SearchEmailOptions{
Expand Down
1 change: 0 additions & 1 deletion routers/web/admin/notice.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ const (
// Notices show notices for admin
func Notices(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.notices")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminNotices"] = true

total := system_model.CountNotices()
Expand Down
1 change: 0 additions & 1 deletion routers/web/admin/orgs.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ const (
// Organizations show all the organizations
func Organizations(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.organizations")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminOrganizations"] = true

explore.RenderUserSearch(ctx, &user_model.SearchUserOptions{
Expand Down
1 change: 0 additions & 1 deletion routers/web/admin/packages.go
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ func Packages(ctx *context.Context) {
}

ctx.Data["Title"] = ctx.Tr("packages.title")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminPackages"] = true
ctx.Data["Query"] = query
ctx.Data["PackageType"] = packageType
Expand Down
2 changes: 0 additions & 2 deletions routers/web/admin/repos.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ const (
// Repos show all the repositories
func Repos(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.repositories")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminRepositories"] = true

explore.RenderRepoSearch(ctx, &explore.RepoSearchOptions{
Expand Down Expand Up @@ -67,7 +66,6 @@ func DeleteRepo(ctx *context.Context) {
// UnadoptedRepos lists the unadopted repositories
func UnadoptedRepos(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.repositories")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminRepositories"] = true

opts := db.ListOptions{
Expand Down
69 changes: 2 additions & 67 deletions routers/web/admin/runners.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,75 +4,10 @@
package admin

import (
"net/url"

actions_model "code.gitea.io/gitea/models/actions"
"code.gitea.io/gitea/models/db"
"code.gitea.io/gitea/modules/base"
"code.gitea.io/gitea/modules/context"
"code.gitea.io/gitea/modules/setting"
actions_shared "code.gitea.io/gitea/routers/web/shared/actions"
)

const (
tplRunners base.TplName = "admin/runners/base"
tplRunnerEdit base.TplName = "admin/runners/edit"
)

// Runners show all the runners
func Runners(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("actions.runners")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminRunners"] = true

page := ctx.FormInt("page")
if page <= 1 {
page = 1
}

opts := actions_model.FindRunnerOptions{
ListOptions: db.ListOptions{
Page: page,
PageSize: 100,
},
Sort: ctx.Req.URL.Query().Get("sort"),
Filter: ctx.Req.URL.Query().Get("q"),
}

actions_shared.RunnersList(ctx, tplRunners, opts)
}

// EditRunner show editing runner page
func EditRunner(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("actions.runners.edit_runner")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminRunners"] = true

page := ctx.FormInt("page")
if page <= 1 {
page = 1
}

actions_shared.RunnerDetails(ctx, tplRunnerEdit, page, ctx.ParamsInt64(":runnerid"), 0, 0)
}

// EditRunnerPost response for editing runner
func EditRunnerPost(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("actions.runners.edit")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminRunners"] = true
actions_shared.RunnerDetailsEditPost(ctx, ctx.ParamsInt64(":runnerid"), 0, 0,
setting.AppSubURL+"/admin/runners/"+url.PathEscape(ctx.Params(":runnerid")))
}

// DeleteRunnerPost response for deleting a runner
func DeleteRunnerPost(ctx *context.Context) {
actions_shared.RunnerDeletePost(ctx, ctx.ParamsInt64(":runnerid"),
setting.AppSubURL+"/admin/runners/",
setting.AppSubURL+"/admin/runners/"+url.PathEscape(ctx.Params(":runnerid")),
)
}

func ResetRunnerRegistrationToken(ctx *context.Context) {
actions_shared.RunnerResetRegistrationToken(ctx, 0, 0, setting.AppSubURL+"/admin/runners/")
func RedirectToDefaultSetting(ctx *context.Context) {
ctx.Redirect(setting.AppSubURL + "/admin/actions/runners")
}
5 changes: 0 additions & 5 deletions routers/web/admin/users.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ const (
// Users show all the users
func Users(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.users")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminUsers"] = true

extraParamStrings := map[string]string{}
Expand Down Expand Up @@ -80,7 +79,6 @@ func Users(ctx *context.Context) {
// NewUser render adding a new user page
func NewUser(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.users.new_account")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminUsers"] = true
ctx.Data["DefaultUserVisibilityMode"] = setting.Service.DefaultUserVisibilityMode
ctx.Data["AllowedUserVisibilityModes"] = setting.Service.AllowedUserVisibilityModesSlice.ToVisibleTypeSlice()
Expand All @@ -102,7 +100,6 @@ func NewUser(ctx *context.Context) {
func NewUserPost(ctx *context.Context) {
form := web.GetForm(ctx).(*forms.AdminCreateUserForm)
ctx.Data["Title"] = ctx.Tr("admin.users.new_account")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminUsers"] = true
ctx.Data["DefaultUserVisibilityMode"] = setting.Service.DefaultUserVisibilityMode
ctx.Data["AllowedUserVisibilityModes"] = setting.Service.AllowedUserVisibilityModesSlice.ToVisibleTypeSlice()
Expand Down Expand Up @@ -253,7 +250,6 @@ func prepareUserInfo(ctx *context.Context) *user_model.User {
// EditUser show editing user page
func EditUser(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("admin.users.edit_account")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminUsers"] = true
ctx.Data["DisableRegularOrgCreation"] = setting.Admin.DisableRegularOrgCreation
ctx.Data["DisableMigrations"] = setting.Repository.DisableMigrations
Expand All @@ -271,7 +267,6 @@ func EditUser(ctx *context.Context) {
func EditUserPost(ctx *context.Context) {
form := web.GetForm(ctx).(*forms.AdminEditUserForm)
ctx.Data["Title"] = ctx.Tr("admin.users.edit_account")
ctx.Data["PageIsAdmin"] = true
ctx.Data["PageIsAdminUsers"] = true
ctx.Data["DisableMigrations"] = setting.Repository.DisableMigrations
ctx.Data["AllowedUserVisibilityModes"] = setting.Service.AllowedUserVisibilityModesSlice.ToVisibleTypeSlice()
Expand Down
78 changes: 0 additions & 78 deletions routers/web/org/org_runners.go

This file was deleted.

4 changes: 0 additions & 4 deletions routers/web/org/setting.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,6 @@ const (
tplSettingsHooks base.TplName = "org/settings/hooks"
// tplSettingsLabels template path for render labels settings
tplSettingsLabels base.TplName = "org/settings/labels"
// tplSettingsRunners template path for render runners settings
tplSettingsRunners base.TplName = "org/settings/runners"
// tplSettingsRunnersEdit template path for render runners edit settings
tplSettingsRunnersEdit base.TplName = "org/settings/runners_edit"
)

// Settings render the main settings page
Expand Down
Loading