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

Security: OpenSSL::SSL::VERIFY_NONE and some http-only requests #249

Open
rmoriz opened this issue Oct 28, 2017 · 1 comment
Open

Security: OpenSSL::SSL::VERIFY_NONE and some http-only requests #249

rmoriz opened this issue Oct 28, 2017 · 1 comment

Comments

@rmoriz
Copy link

rmoriz commented Oct 28, 2017

  • Currently all https/TLS requests are vulnerable to MITM.

    See:

    if @ssl
    http.use_ssl = true
    http.verify_mode = OpenSSL::SSL::VERIFY_NONE
    end

    def get_artifact(job_name,filename)
    @artifact = job.find_artifact(job_name)
    uri = URI.parse(@artifact)
    http = Net::HTTP.new(uri.host, uri.port)
    http.verify_mode = OpenSSL::SSL::VERIFY_NONE

    Suggestion: Change default to verify, allow users who are unable to fix their trust root to set an option to disable verification.

  • exec_cli is hard-coded to http:

    def exec_cli(command, args = [])
    base_dir = File.dirname(__FILE__)
    server_url = "http://#{@server_ip}:#{@server_port}/#{@jenkins_path}"

  • update center request is not made over https, too.

    @logger.info "Initializing Jenkins Update Center..."
    @logger.debug "Obtaining the JSON data for Update Center..."
    # TODO: Clean me up
    update_center_data = open("http://updates.jenkins-ci.org/update-center.json").read

@rmoriz
Copy link
Author

rmoriz commented Oct 30, 2017

see #204

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

No branches or pull requests

1 participant