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

Added the option SkipCertificateCheck for self-signed certificates #46

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

Conversation

Frickeldave
Copy link

Hi,

based on issue #45 i added the option "SkipCertificateCheck" to all functions. I hope it's all fine, i am not the god-mode-powersheller. Let me know if something must changed.
I tested it against a RabbitMQ instance with Version 3.9.1. with the alpine-management container. The self-signed certicates are created with openssl.

SSLSUBJECT="/C=$CRT_C/ST=$CRT_S/L=$CRT_L/O=$CRT_OU/CN=$CRT_CN"
openssl req -x509 -newkey rsa:${CRT_LENGTH} -keyout /home/appuser/data/certificates/key.key -out /home/appuser/data/certificates/cer.crt -days $CRT_VALIDITY -nodes -subj "$SSLSUBJECT"

@Frickeldave
Copy link
Author

Hey,

any issues with this change? would be great to get that merged.

Regards

Dave

@Frickeldave
Copy link
Author

Hey,

again, just to give you ping. Would be great to merge this stuff into your repo. Please let me know, if i can support you by doing this...

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.

1 participant