blueriver/MuraCMS — pull request #1679 by nsollecito (2014-12-01 20:03:50+00:00)
Title: Fixing conditional setLog logic

I found 2 issues with Portcullis.cfc. First, updateLog() never runs properly because conditional is explicitly checking for value of isLogged to be 1 though return value is "Yes" or "No". As a result, IP address is always appended to the log and nothing ever gets blocked.
Second, the IP Address from the arguments scope isn't referenced and uses Mura's request.remoteAddr instead which will cause different IP addresses to be logged which just compounds this problem.

I want to do some more testing with this, however, the IP logging / blocking functionality is busted in it's current state and we want to leverage this setLog / blocking functionality outside of the default scan() implementation