Add Support ServerGenerateOneTimeTokenRequestResolver

[franticticktick]

Closes gh-16488

[jzheaux]

Hi, @franticticktick. This could be reasonable to add. Can you explain more about your use case so that we are on the same page as to the problem you are wanting to solve?

[jzheaux]

Please disregard that message, I wasn't aware that #16297 had been merged.

[jzheaux]

Thanks, @franticticktick! I left my feedback inline.

[franticticktick]

Hi @jzheaux, thnaks for your feedback. I modified the OneTimeTokenLoginSpecTests according to the pattern that @rwinch made. There is only one point that we need to pay attention to, I noted it in your comments. Could you please review the changes?

[jzheaux]

Great, @franticticktick. Sorry that this specific request was unclear: Will you please ensure that the concurrency repair is in a separate commit that precedes your feature enhancement? This will make it easier to compare that change to Rob's change and also make it easier to backport, should that become necessary.

[franticticktick]

I didn't look at your comment carefully, sorry please. I made two commits and also removed the unstable expiration check as @rwinch did. I made similar changes to ServerOneTimeTokenLoginDslTests.

[jzheaux]

Thanks, @franticticktick! This is now merged into main. And thanks again for separating the concurrency fix into a separate commit; I realize we usually ask for commits to be squashed and that was extra work for you to tease them apart.