Skip to content

Conversation

@alexpop
Copy link
Collaborator

@alexpop alexpop commented Sep 27, 2016

Listening ports can be bound on any interface available on the node. If this is the case, the ssl resource needs to use that IP address and not localhost or target IP.

Copy link
Member

@arlimus arlimus left a comment

Choose a reason for hiding this comment

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

Also I'm not sure if we need the Debug resource; how do you envision it?

Copy link
Member

Choose a reason for hiding this comment

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

Maybe as a different suggestion: (still not happy here)

invalid_addrs = %w{ 127.0.0.1 0.0.0.0 ::1 :: } tcpports = ... enabled_sockets = ... ssl_list = enabled_sockets.map do |s| params = { port: s.port } params[:host] = s.address unless invalid_addrs.include?(s.address) [s, ssl(params)] end control 'tls1.2' do title 'Run TLS 1.2 whenever SSL is active on a port' impact 0.5 ssl_list.each do |socket, ssl_socket| # create a description proc_desc = "on node == #{command('hostname').stdout.strip} running #{socket.process.inspect} (#{socket.pid})" describe ssl_socket.protocols('tls1.2') do it(proc_desc) { should be_enabled } it { should be_enabled } end end end 
@alexpop alexpop force-pushed the ap/target-addresses branch from bcb9285 to 88d54ef Compare October 10, 2016 16:56
@alexpop alexpop force-pushed the ap/target-addresses branch from 88d54ef to 12b8fcf Compare October 10, 2016 16:59
@alexpop
Copy link
Collaborator Author

alexpop commented Oct 10, 2016

Thanks for the feedback Dom.
I updated my branch to handle the host without the lambda method.

Also getting the hostname only once instead of 6 times per port.

I've also made a few more cosmetic changes.

I would keep the debugging control until we iron out all the edge cases.

@alexpop alexpop changed the title WIP: target addresses of listening ports and add control for troubleshooting target addresses of listening ports and add control for troubleshooting Oct 10, 2016
@alexpop
Copy link
Collaborator Author

alexpop commented Oct 10, 2016

screen shot 2016-10-10 at 6 24 39 pm

^ sample scan with this profile

@chris-rock
Copy link
Member

@arlimus Could you please re-review @alexpop updates?

@chris-rock
Copy link
Member

Thank you @alexpop

@chris-rock chris-rock merged commit 71a2051 into master Feb 3, 2017
@chris-rock chris-rock deleted the ap/target-addresses branch February 3, 2017 07:40
@supergicko supergicko mentioned this pull request Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants