Allow periods in the external_id value used in the /users/by-external route. (PR #3163)

I’m not at all a Rubyist, so I’m not sure if there was a better way to do this. If nothing else, I hope it can at least inspire a correct solution.


Thanks for contributing this pull request! Could you please sign our CLA so we can review it?

CLA signed.

I want to merge this in but it needs a rebase

ping @rcfox

Hey, sorry about that. The merge seemed pretty simple, but I’m not really able to test it at the moment.

we should pull this in, or synthesize the fix ourselves, unless someone has a compelling reason not to

Ill pull this in for now