esgf-pyclient icon indicating copy to clipboard operation
esgf-pyclient copied to clipboard

added username to logon_with_openid to script CEDA access more easily

Open cojacoo opened this issue 4 years ago • 1 comments

Hi, Thank you so much for this package. I ran quite sometimes into a minor issue with the logon function when CEDA actually required a username but self._get_logon_details(openid) would not provide them. I suggest a very small change to allow for explicit username provision to the logon_with_openid function. If left empty, everything will work as before. If provided, the username is propagated to the logon function. This fix solves the issue for me. All the best. Conrad

cojacoo avatar Dec 20 '19 10:12 cojacoo

Hi @cojacoo , the change looks fine but I would rather we had an extra unit test or two to support it. Could you add the test(s) and include them in the Pull Request? Many thanks

agstephens avatar Jan 02 '20 14:01 agstephens