Skip to content

Use range for iteration values and use switch instead of if chain#2

Open
kkirsche wants to merge 1 commit intormasci:masterfrom
kkirsche:patch-1
Open

Use range for iteration values and use switch instead of if chain#2
kkirsche wants to merge 1 commit intormasci:masterfrom
kkirsche:patch-1

Conversation

@kkirsche
Copy link

@kkirsche kkirsche commented Jun 8, 2016

Use range for iteration values and use switch instead of if chain

Use range for iteration values and use switch instead of if chain
@rmasci
Copy link
Owner

rmasci commented Jun 9, 2016

Hi Kevin,

Thanks for fixing this up! I am not a programmer by trade, I wrote these
because I needed something to do ssl queries without a DNS which allows you
to check the ssl cert of a system that is behind a load balancer. I also
like the ssl package you wrote!!

Thanks,

Rich

On Wed, Jun 8, 2016 at 4:51 PM, Kevin Kirsche notifications@github.com
wrote:

Use range for iteration values and use switch instead of if chain

You can view, comment on, or merge this pull request online at:

#2
Commit Summary

  • Use range for iteration values and use switch instead of if chain

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#2, or mute the thread
https://github.com/notifications/unsubscribe/AA0UPUhBe48xHsJ_cJJNUgW4jOuD7LIpks5qJzlYgaJpZM4IxcKV
.

@kkirsche
Copy link
Author

kkirsche commented Jun 9, 2016

Course. I generally try to show things like this as you never know if the other person has experience with these things. I know I've had people submit pull requests and it makes me just go, wow I wish I had known that when I started. Without sharing there isn't a way to learn though. Glad you found it helpful and let me know if you have any questions. I found your tool after I started mine as I needed to check SSL Versions for work and vulnerability management of systems.

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.

2 participants