knotserver: add owners to known_dids #531

merged
opened by oppi.li targeting master from push-lnvnxmxlqkux

this is a regression from the v1.8.0 release

Signed-off-by: oppiliappan me@oppi.li

Changed files
+14 -4
knotserver
+14 -4
knotserver/routes.go
···
}
// remove existing owner
-
err = h.e.RemoveKnotOwner(rbacDomain, existingOwner)
-
if err != nil {
-
return nil
+
if err = h.db.RemoveDid(existingOwner); err != nil {
+
return err
}
+
if err = h.e.RemoveKnotOwner(rbacDomain, existingOwner); err != nil {
+
return err
+
}
+
default:
return fmt.Errorf("more than one owner in DB, try deleting %q and starting over", h.c.Server.DBPath)
}
-
return h.e.AddKnotOwner(rbacDomain, cfgOwner)
+
if err = h.db.AddDid(cfgOwner); err != nil {
+
return fmt.Errorf("failed to add owner to DB: %w", err)
+
}
+
if err := h.e.AddKnotOwner(rbacDomain, cfgOwner); err != nil {
+
return fmt.Errorf("failed to add owner to RBAC: %w", err)
+
}
+
+
return nil
}