fix: have GetModuleContext use config and secrets from the context in…

…stead of creating a new managers using the envar file
block · May 20, 2024 · 339dddf · 339dddf
1 parent 9fd04bf
commit 339dddf
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/backend/controller/controller.go b/backend/controller/controller.go
@@ -657,18 +657,13 @@ nextModule:
 func (s *Service) GetModuleContext(ctx context.Context, req *connect.Request[ftlv1.ModuleContextRequest]) (*connect.Response[ftlv1.ModuleContextResponse], error) {
 	name := req.Msg.Module
 
-	cm, err := cf.NewDefaultConfigurationManagerFromConfig(ctx, cf.ConfigFromEnvironment())
-	if err != nil {
-		return nil, connect.NewError(connect.CodeInternal, fmt.Errorf("could not get configs: %w", err))
-	}
+	cm := cf.ConfigFromContext(ctx)
+	sm := cf.SecretsFromContext(ctx)
+
 	configs, err := cm.MapForModule(ctx, name)
 	if err != nil {
 		return nil, connect.NewError(connect.CodeInternal, fmt.Errorf("could not get configs: %w", err))
 	}
-	sm, err := cf.NewDefaultSecretsManagerFromConfig(ctx, cf.ConfigFromEnvironment())
-	if err != nil {
-		return nil, connect.NewError(connect.CodeInternal, fmt.Errorf("could not get secrets: %w", err))
-	}
 	secrets, err := sm.MapForModule(ctx, name)
 	if err != nil {
 		return nil, connect.NewError(connect.CodeInternal, fmt.Errorf("could not get secrets: %w", err))