diff --git a/controllers/ext_client.go b/controllers/ext_client.go index 4408c624..0f1f3db3 100644 --- a/controllers/ext_client.go +++ b/controllers/ext_client.go @@ -430,14 +430,14 @@ func createExtClient(w http.ResponseWriter, r *http.Request) { extclient.Enabled = parentNetwork.DefaultACL == "yes" } - if err := logic.SetClientDefaultACLs(&extclient); err != nil { - slog.Error("failed to set default acls for extclient", "user", r.Header.Get("user"), "network", node.Network, "error", err) + if err = logic.CreateExtClient(&extclient); err != nil { + slog.Error("failed to create extclient", "user", r.Header.Get("user"), "network", node.Network, "error", err) logic.ReturnErrorResponse(w, r, logic.FormatError(err, "internal")) return } - if err = logic.CreateExtClient(&extclient); err != nil { - slog.Error("failed to create extclient", "user", r.Header.Get("user"), "network", node.Network, "error", err) + if err := logic.SetClientDefaultACLs(&extclient); err != nil { + slog.Error("failed to set default acls for extclient", "user", r.Header.Get("user"), "network", node.Network, "error", err) logic.ReturnErrorResponse(w, r, logic.FormatError(err, "internal")) return } diff --git a/migrate/migrate.go b/migrate/migrate.go index 92245a12..ce4cc840 100644 --- a/migrate/migrate.go +++ b/migrate/migrate.go @@ -284,9 +284,11 @@ func updateAcls() { } // save new acls + slog.Info(fmt.Sprintf("(migration) saving new acls for network: %s", network.NetID), "networkAcl", networkAcl) if _, err := networkAcl.Save(acls.ContainerID(network.NetID)); err != nil { slog.Error(fmt.Sprintf("error during acls migration. error saving new acls for network: %s", network.NetID), "error", err) continue } + slog.Info(fmt.Sprintf("(migration) successfully saved new acls for network: %s", network.NetID)) } }