I've just started reviewing this, and the first thing I notice is that the implementation doesn't use the existing ACL/ACE security machinery. Is there a reason for that?
--- ** [tickets:#6392] Per tool user bans** **Status:** code-review **Labels:** support p3 42cc **Created:** Tue Jun 25, 2013 02:05 PM UTC by Chris Tsai **Last Updated:** Wed Sep 04, 2013 02:06 PM UTC **Owner:** nobody It should be possible to ban users from posting to individual tools, in case of spam, abuse, etc. There should also be a spot to indicate the reason when banning. These bans should be collected in a site or neighborhood wide admin page so we can review these periodically in case we want to remove/disable that account site wide. --- Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.