Mutating request.META["REMOTE_ADDR"] doesn't seem like the best approach to me. In fact, why have this in middleware at all? Since we're only talking about using it in [one place](https://github.com/mozilla/fxoss/blob/master/protected_assets/page_processors.py#L22), we could just move this logic inline to the view, but assign the value from parts[0] to the Agreement.ip directly. If we do need similar logic in multiple places, I'd rather see it as a utility function returning the ip called only from the views that need it.