Avoid default (_) case in match

[obycode]

          I don't like this function, it should never use `_ => true` (it should explicitly handle all cases), because if we ever add to these enums it will still build and return incorrect values

Originally posted by @jbencin in #5630 (comment)

[jcnelson]

From sync call:

Let's see if there's a clippy lint to check for this, and add it to CI.

[jferrant]

From sync call:

Let's see if there's a clippy lint to check for this, and add it to CI.

I think its called wildcard_match_arm

[jbencin]

I don't think using wildcard is always bad, just in this particular case _ => true fails in the wrong direction. It would return true for new/unknown states, where we would want to return false

It would be a huge task to apply this entire lint to entire repository, but maybe it should be applied to certain files, like auth.rs