search

ucphhpc / migrid-sync

MiGrid workspace where master branch is kept strictly in sync with SF upstream svn repo. Any development or experiments should use a branch. You probably want to fork your own clone or work e.g. on the edge branch if you wish to contribute.
GNU General Public License v2.0
3 stars 4 forks source link

External account renewal link and failed peers_email validation #20

Closed SebastianPrehn closed 9 months ago

SebastianPrehn commented 11 months ago

In some instances where external users try to connect via the links provided in the account expire mail, they get the following error: input parsing error: peers_email: <email>: No actual email address included Input arguments were rejected - not allowed for this script!

jonasbardino commented 10 months ago

Thanks for reporting @SebastianPrehn. I adjusted the title a bit to remember what it's about. The failing link is the first one in the expire warning email for logging in and renewing before the account actually expires. Login succeeds but then the twofactor auth step fails with the above error because the @ ends up being url-encoded there, and therefore it doesn't fit our email validation. It basically needs further tweaking after we introduced explicit peers_email passing as part of that link.

jonasbardino commented 9 months ago

It should be fixed in the code (svn rev5843) now and applied on ERDA

jonasbardino commented 8 months ago

We got reports about a similar error when using a Peer with Danish characters in the name. It is resolved in svn rev 5884 and the corresponding git revisions.