search

fermitools / cigetcert

Get an X.509 certificate with SAML ECP and store proxies.
Other
2 stars 6 forks source link

Updates for python3 syntax #49

Closed duncanmmacleod closed 5 years ago

duncanmmacleod commented 5 years ago

This PR updates the cigetcert script to use python3-compatible syntax for print and except.

DrDaveD commented 5 years ago

Thanks, Duncan, looks good.

DrDaveD commented 5 years ago

This is now released in tagged version 1.17.

duncanmmacleod commented 5 years ago

Thanks @DrDaveD. This isn't actually enough for the script to work under python3. I have a few more fixes to put in that I've tested on my machine. Would you rather I put in one PR with everything else, or keep them in smaller, logical chunks?

For context, I was put onto cigetcert by @paulhopkins as a possible solution to creating an X509 proxy certificate for LIGO authentication on Windows, but it looks like a good wholesale replacement for our Globus-based proxy creator.

DrDaveD commented 5 years ago

I can go either way, whichever you prefer.

I have long thought that cigetcert would be good for LIGO, I am glad somebody is working on it.

duncanmmacleod commented 5 years ago

@DrDaveD, ok, I'll start posting some PRs. I also think it would be useful to have this as a full python library, mainly to support pip installations. I can also put that together if you would be open to it.

DrDaveD commented 5 years ago

Sure I would be open to that.