cyclops1982 / SourceCode.ServiceBroker.RolesManagement

A service broker that allows you to manage roles.
1 stars 0 forks source link

Delete Role Item does not work when Security Label is supplied #9

Open pblommendaal opened 12 years ago

pblommendaal commented 12 years ago

When the RoleItem prop is supplied with the security label like: K2:\domainname\peter the Delete Role Item does not work, but it doesn't return an error either.

Can we fix this so the Security Label can be supplied?

cyclops1982 commented 12 years ago

Peter,

I think the security label can(and must?) be supplied. It won't match if it's not supplied, or that's how i think it should work.

For issue #5 you would want the item to ignore faulty input, and here you want a exception? I'm not sure how to handle this, as i think the behaviour should be the same for all methods.

cyclops1982 commented 12 years ago

@pblommendaal Can you provide some more input on this item. If i don't supply a security label - it does not work. IMHO this is how it should be (or we should add a default security label). Please note that a security label is K2:DOMAIN\user - not K2:\DOMAIN\user (not the slashes)

pblommendaal commented 12 years ago

I thought the SEC label should be supplied. Will test again tomorrow with SO tester and smartforms.

cyclops1982 commented 12 years ago

Ah, please note that what you must provide is a FQN. This is with the security label which actually is 'K2' (not the full thing, as that's a FQN). That might be confusing in my previous comment.