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

Add TLSConfig to nctalk #1195

Merged
merged 1 commit into from Aug 30, 2020
Merged

Conversation

gary-kim
Copy link
Contributor

Still need to test. Give me a bit to set up an instance with improper SSL config.

Signed-off-by: Gary Kim gary@garykim.dev

@gary-kim gary-kim force-pushed the enh/noid/disable-tls-nctalk branch 2 times, most recently from b50480b to 243a7cd Compare August 13, 2020 13:45
@gary-kim gary-kim marked this pull request as draft August 14, 2020 13:23
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 12753 lines exceeds the maximum allowed for the inline comments feature.

@gary-kim gary-kim marked this pull request as ready for review August 28, 2020 21:27
@gary-kim
Copy link
Contributor Author

Confirmed to be working

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 12753 lines exceeds the maximum allowed for the inline comments feature.

Signed-off-by: Gary Kim <gary@garykim.dev>
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 12753 lines exceeds the maximum allowed for the inline comments feature.

@codeclimate
Copy link

codeclimate bot commented Aug 28, 2020

Code Climate has analyzed commit 7b9b18e and detected 0 issues on this pull request.

View more on Code Climate.

@42wim 42wim added this to the 1.18.3 milestone Aug 30, 2020
@42wim
Copy link
Owner

42wim commented Aug 30, 2020

Thanks 👍

@42wim 42wim merged commit a0741d9 into 42wim:master Aug 30, 2020
@gary-kim gary-kim deleted the enh/noid/disable-tls-nctalk branch August 30, 2020 12:07
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.

None yet

3 participants