Re: [PATCH 1/2] Added new auth policy blacklist.

On Sat, Oct 19, 2013 at 12:46:50AM +0400, Alexander Chemeris wrote:

Well, ACCEPT_ALL is accept all - a different thing :)

yes it is. But what does ACCEPT_NEW (blacklist) take away from ACCEPT_ALL? My argument is that ACCEPT_NEW is the better "ACCEPT"

I would rather rename BLACKLIST to ACCEPT_NEW and make it default instead of ACCEPT_ALL. It will give better experience to a random OsmoNITB user (user in terms of an NITB operator, not in terms of a subscriber).

Well, closed should be the default. We want to minizime the interference people can cause by default.