github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #6686 from dokku/k3s-autoscaling-validation

Browse Source 
Validate input for the autoscaling-auth:set command
This commit is contained in:
Jose Diaz-Gonzalez
2024-03-08 06:21:23 -05:00
committed by GitHub
parent 4d18c8d9d9 8f5a834cf0
commit bfe552ee43
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
plugins/scheduler-k3s/subcommands.go
View File

@@ -64,6 +64,16 @@ func CommandAutoscalingAuthSet(appName string, trigger string, metadata map[stri
appName = "--global"
}
if appName != "--global" {
if err := common.VerifyAppName(appName); err != nil {
return err
}
}
if len(trigger) == 0 {
return fmt.Errorf("Missing trigger type argument")
}
if len(metadata) == 0 {
properties, err := common.PropertyGetAllByPrefix("scheduler-k3s", appName, fmt.Sprintf("%s%s.", TriggerAuthPropertyPrefix, trigger))
if err != nil {
@@ -95,6 +105,9 @@ func CommandAutoscalingAuthSet(appName string, trigger string, metadata map[stri
if err := common.PropertyWrite("scheduler-k3s", appName, fmt.Sprintf("%s%s.%s", TriggerAuthPropertyPrefix, trigger, key), value); err != nil {
return fmt.Errorf("Unable to set property: %w", err)
}
common.LogInfo1("Trigger authentication settings saved")
common.LogVerbose("Resources will be created or updated on next deploy")
}
if appName == "--global" {