Would it be better if these three lines were "inside" the `||=` so they are only executed once? I think every `active?` fires off a request to Redis, so this could potentially be an "expensive" query?