towk/configurator
1
0
Fork 0
mirror of https://github.com/davidallendj/configurator.git synced 2025-12-20 11:37:00 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

generator: added warn when default generator not found and fix error messages

Browse source
This commit is contained in:
David Allen 2024-11-14 13:45:19 -07:00
parent e93bef79f2
commit 2a9e7c72dc
Signed by: towk
GPG key ID: 793B2924A49B3A3F
2 changed files with 7 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
pkg/generator/generator.go
View file

@ -12,6 +12,7 @@ import (
"github.com/OpenCHAMI/configurator/pkg/util" "github.com/OpenCHAMI/configurator/pkg/util"
"github.com/nikolalohinski/gonja/v2" "github.com/nikolalohinski/gonja/v2"
"github.com/nikolalohinski/gonja/v2/exec" "github.com/nikolalohinski/gonja/v2/exec"
"github.com/rs/zerolog/log"
) )
type ( type (
@ -440,6 +441,7 @@ func GenerateWithTarget(config *configurator.Config, params Params) (FileMap, er
generator, ok = DefaultGenerators[params.Target] generator, ok = DefaultGenerators[params.Target]
if !ok { if !ok {
// only load the plugin needed for this target if we don't find default // only load the plugin needed for this target if we don't find default
log.Error().Msg("did not find target in default generators")
generator, err = LoadPlugin(target.PluginPath) generator, err = LoadPlugin(target.PluginPath)
if err != nil { if err != nil {
return nil, fmt.Errorf("failed to load plugin: %w", err) return nil, fmt.Errorf("failed to load plugin: %w", err)

10
pkg/server/server.go
View file

@ -76,7 +76,7 @@ func (s *Server) Serve() error {
var err error var err error
tokenAuth, err = configurator.FetchPublicKeyFromURL(s.Config.Server.Jwks.Uri) tokenAuth, err = configurator.FetchPublicKeyFromURL(s.Config.Server.Jwks.Uri)
if err != nil { if err != nil {
logrus.Errorf("failed to fetch JWKS: %w", err) logrus.Errorf("failed to fetch JWKS: %v", err)
continue continue
} }
break break
@ -138,7 +138,7 @@ func (s *Server) Generate(w http.ResponseWriter, r *http.Request) {
// generate a new config file from supplied params // generate a new config file from supplied params
outputs, err := generator.GenerateWithTarget(s.Config, s.GeneratorParams) outputs, err := generator.GenerateWithTarget(s.Config, s.GeneratorParams)
if err != nil { if err != nil {
writeErrorResponse(w, "failed to generate file: %w", err) writeErrorResponse(w, "failed to generate file: %v", err)
return return
} }
@ -146,12 +146,12 @@ func (s *Server) Generate(w http.ResponseWriter, r *http.Request) {
tmp := generator.ConvertContentsToString(outputs) tmp := generator.ConvertContentsToString(outputs)
b, err := json.Marshal(tmp) b, err := json.Marshal(tmp)
if err != nil { if err != nil {
writeErrorResponse(w, "failed to marshal output: %w", err) writeErrorResponse(w, "failed to marshal output: %v", err)
return return
} }
_, err = w.Write(b) _, err = w.Write(b)
if err != nil { if err != nil {
writeErrorResponse(w, "failed to write response: %w", err) writeErrorResponse(w, "failed to write response: %v", err)
return return
} }
} }
@ -163,7 +163,7 @@ func (s *Server) Generate(w http.ResponseWriter, r *http.Request) {
func (s *Server) ManageTemplates(w http.ResponseWriter, r *http.Request) { func (s *Server) ManageTemplates(w http.ResponseWriter, r *http.Request) {
_, err := w.Write([]byte("this is not implemented yet")) _, err := w.Write([]byte("this is not implemented yet"))
if err != nil { if err != nil {
writeErrorResponse(w, "failed to write response: %w", err) writeErrorResponse(w, "failed to write response: %v", err)
return return
} }
} }