Remove getter-- since this is just a property
Cleanup "rights" function

Since a success followed by a failure is still a success, there is no reason to check the rest

In addition, consolidating the logic to remove the local "good" since it isn't needed
Merge pull request #19037 from jacksontj/develop

Misc. eauth cleanup