Title: #250: cache_req: Use ternary operator instead of bool+switch
Personally I think any of the suggestions make the code less readable than the switch
(which as all the cases explicitly matching with the comments above).
For me, it's a nack for the patch and for those suggestions.
See the full comment at https://github.com/SSSD/sssd/pull/250#issuecomment-297676645