Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.0] Update dirac_cert_convert.py, adding cmd-line option "--legacy" to allow pcks12 certificate with legacy crypto #7419

Closed
wants to merge 1 commit into from

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Jan 24, 2024

Replaces #7418

Modified dirac_cert_convert.py to allow pcks12 certificate with legacy crypto

Some providers (cf. CNRS) use certificates with outdated crypto, so need to call openssl with the "-legacy" option.

Modified dirac_cert_convert.py to add a switch for this ("--legacy") using the @script mechanisms.

Added the long option only, not putting the short option ("-l") since this should be exceptional.

Also added a gLogger.warn() for this.

BEGINRELEASENOTES

  • Framework
    FIX: Modified dirac_cert_convert.py to add a switch for legacy certificates

ENDRELEASENOTES

…legacy crypto

Some providers (cf. CNRS) use certificates with outdated crypto,
so need to call `openssl` with the "-legacy" option.

Modified `dirac_cert_convert.py` to add a switch for this ("--legacy") using the `@Script` mechanisms.

Added the long option only, not putting the short option ("-l") since this should be exceptional.

Also added a gLogger.warn() for this.
@fstagni fstagni force-pushed the certConvertLegacyF branch from 4a0d4ab to 35f232b Compare January 24, 2024 09:26
@fstagni
Copy link
Contributor Author

fstagni commented Jan 25, 2024

Closing, as per #7381 (comment)

@fstagni fstagni closed this Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants