Mailinglist Archive: opensuse-buildservice (258 mails)

[Marcus Hüwe <suse-tux@xxxxxx>]

On 2010-08-08 14:15:41 -0600, Srinidhi B wrote:
>  >>> On Monday, 09 August, 2010 at 01:34 AM, Marcus Hüwe<suse-tux@xxxxxx> 
> wrote: 
> > Hi,
> >
> > On 2010-08-08 13:38:45 -0600, Srinidhi B wrote:
> > > If you try to delete a user from the api (Rails app), you will get a 
> > message saying:
> > >     "You sent an invalid request!"
> > >
> > > After looking at api/app/controllers/active_rbac/user_controller.rb, I 
> > found that you can only POST to destroy and can only GET on delete. 
> > > So here's another trivial patch that allows deletion of users from the 
> > Rails app.
> > >
> > User deletion doesn't "work" at the moment. The issue with this patch is 
> > that
> > the "destroy" method just removes the user from the database and doesn't 
> > update
> > the project/package metadata relations => the db is an "inconsistent" state.
>
> Ah okay! I understand. I ensured that I deleted those user's home projects 
> before deleting the user - and I knew that those users weren't part of any 
> other projects / packages. :) 
>
> So, the correct fix would be to update all projects / packages to which this 
> user belongs and only then delete the user from the db? Or should there be 
> some more checks?
Hmm yes + the groups_users table. Additionally the backend needs to be informed
too otherwise the user would still be in the project/package metadata.


Marcus
-- 
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx