-
Notifications
You must be signed in to change notification settings - Fork 1
NoTLS option #minor #21
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just noticed: https://github.com/neicnordic/sda-mq/blob/master/README.md?plain=1#L3 might require a change as well
Adds the NOTLS env to disable SSL completely
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
not sure we want to remove hooks
folder with this or another PR
This is not needed anymore since Docker Hub is not building images any longer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, although I didn't test it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't figure out how test it, but as far as I can tell it looks good!
Describe the pull request:
Pull request long description:
Changes made:
Related issues:
Additional information:
Release note:
Documentation change:
Mentions: