Prevent redirecting again in the middle of redirect (ORCID#1926)

[kadet1090]

This should hopefully be temporary fix for issue mentioned in #1926 where some of SSO clients like OSF or our Bridge of Knowledge have problems with users being unable to log in into the application. The issue is caused by the race condition of GTM and GTM timeout (4s) so as temporary solution I propose to simply allow only one call to the sendUserToRedirectURL which should fix the problem.

However proper solution is needed because setting 4s timeout as workaround for users that have addons blocking GTM does not seem like proper solution causing unnecessary delay for some users.

[leomendoza123]

The problem is that we have 2 redirects, one is called by GTM library it self, and the other one is called by sendUserToRedirectURL.

So in reality the sendUserToRedirectURL is only call once, and that is why this boolean check does not work .

[leomendoza123]

@kadet1090 We also agree, adding a 4 seconds delay is unfair to our users with add blockers.
And this 4 seconds delay should be consider only a temporal fix.

I have create a Trello ticket to followup with this issue on our boards
https://trello.com/c/gCAPBN4M/8576-qa-users-with-add-blocks-have-to-wait-4-extra-seconds-on-oauth-process

[amontenegro]

Thanks a lot for your contribution @kadet1090! We found another way of fixing this and we are thinking about how to remove the 4 secs delay.

Thanks