I don't like that `sid_collision?` now mutates redis.  Can the `setnx` be moved outside of `sid_collision?`?  If not, I think the method name should be changed to something like `claim_sid_unless_collision`.