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

Fix OpenSSL 3 breaks PKCS12 tmp file generation #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

satmandu
Copy link

Fixes #53

  • Add check for OpenSSL 3.x, and have OpenSSL use the -legacy flag when generating the PKCS12 temp file if OpenSSL 3.x is being used.

@videoman
Copy link

My cronjob screwed up my install of the UniFi Controller cause of this. Would be good to comment this to the main branch.

@gctwnl
Copy link

gctwnl commented Nov 23, 2022

Is this software still maintained? If people download the script they get the 2017 version without the fix. I was looking into using this script (adapting it) so it can be used when the UC is run inside a docker container. Probably best is so it can update the cert inside a running container (probably requires an additional script)

@satmandu
Copy link
Author

I have been unable to contact the maintainer to get this merged. :/

@satmandu
Copy link
Author

satmandu commented Nov 23, 2022

Maybe someone can try to reach out to him here, or just fork the repo....

@gctwnl
Copy link

gctwnl commented Nov 23, 2022

That link is dead (no LinkedIn page there) so we might consider all of this dead.

I'm using this as inspiration to build my own that works with a docker container I am using.

@satmandu
Copy link
Author

LinkedIn link fixed!

@gctwnl
Copy link

gctwnl commented Nov 23, 2022

I'm going to build my own solution based on his, but running inside the container. Maybe publish that later.

@satmandu
Copy link
Author

Please note your solution here!

@gctwnl
Copy link

gctwnl commented Nov 24, 2022

jacobalberty/unifi-docker#609

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OpenSSL 3 breaks PKCS12 tmp file generation
3 participants