Skip to content

Commit

Permalink
Improve logging for LDAP group creation
Browse files Browse the repository at this point in the history
Without this patch, one might wonder why the ldap group is
not resulting in the creation of a project.

This is a problem, as it reduces the ability to debug.

This fixes it by bubbling up the error.
  • Loading branch information
evrardjp-cagip committed Jan 13, 2025
1 parent acbd644 commit 3241aba
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/project/constructors.go
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ func GetProjectsFromGrouplist(groups []string) []*types.Project {
for _, projectGroup := range groups {
project, err := NewProject(projectGroup)
if err != nil {
slog.Error(fmt.Sprintf("Could not generate project name from group %v", projectGroup))
slog.Error("Could not generate project name from group name", "group", projectGroup, "reason", err)
continue
}
projects = append(projects, project)
Expand Down

0 comments on commit 3241aba

Please sign in to comment.