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
4 stars 4 forks source link

Prevent unique_id user database field problem in GDP mode #127

Closed jonasbardino closed 2 months ago

jonasbardino commented 2 months ago

Fix an issue with the new unique_id user database field inadvertently being passed onto the related GDP project users and then causing failure to e.g. reset password, because that copy would then be identical to the value associated with the main user and detected as an illegal ID collision. Blank out the unique_id during GDP user creation and skip any such existing values in the collision check. Added notes about reconsidering if GDP project users should actually get a unique ID assigned as well in the future as briefly discussed internally.

jonasbardino commented 2 months ago

Manually merged through svn after offline sanity check internally. Feel free to comment or review here, too, @Martin-Rehr

jonasbardino commented 2 months ago

Really merged