Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Minor Minor
    • Resolution: Incomplete
    • Affects Version/s: JRuby 1.6.7, JRuby 1.7.0.pre1
    • Fix Version/s: JRuby 1.7.0.pre2
    • Component/s: Java Integration
    • Labels:
      None
    • Environment:

      jruby -J-Dhttp.proxyHost=localhost -J-Dhttp.proxyPort=8080
    • Number of attachments :
      0

      Description

      There is an accepted http_proxy setting as an environment variable or setting a JVM System property http.proxyHost. Net::HTTP seems to only register a proxy setting when explicitly declared with Net::HTTP::Proxy#start.

      Net::HTTP should check at least the JVM system property when making requests. The doc does mention not using the ENVIRONMENT variable but I believe a special case should be made for a JVM setting.

      https://github.com/jruby/jruby/pull/195

        Activity

        Hide
        Charles Oliver Nutter added a comment -

        I am not opposed. Looking into the PR.

        Show
        Charles Oliver Nutter added a comment - I am not opposed. Looking into the PR.
        Hide
        Charles Oliver Nutter added a comment -

        Discussion is happening on the PR. Since the PR shows up as an issue I'm resolving this one.

        Show
        Charles Oliver Nutter added a comment - Discussion is happening on the PR. Since the PR shows up as an issue I'm resolving this one.

          People

          • Assignee:
            Charles Oliver Nutter
            Reporter:
            Jacob Evans
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1 hour
              1h
              Remaining:
              Remaining Estimate - 1 hour
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified