On 5 November 2013 07:42, Konstantin Kolinko <knst.koli...@gmail.com> wrote:
> 2013/11/4  <ol...@apache.org>:
>> Author: olamy
>> Date: Mon Nov  4 04:12:13 2013
>> New Revision: 1538502
>>
>> URL: http://svn.apache.org/r1538502
>> Log:
>> [MTOMCAT-92] Bind tomcat to a self-chosen network interface
>>
>> Modified:
>>     
>> tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
>>     
>> tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java
>>
>> Modified: 
>> tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
>> URL: 
>> http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java?rev=1538502&r1=1538501&r2=1538502&view=diff
>> ==============================================================================
>> --- 
>> tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
>>  (original)
>> +++ 
>> tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
>>  Mon Nov  4 04:12:13 2013
>> @@ -136,6 +136,16 @@ public abstract class AbstractRunMojo
>>      private int port;
>>
>>      /**
>> +     *
>> +     * For servers with more than one IP address, this attribute specifies 
>> which address
>> +     * will be used for listening on the specified port.
>> +     * By default, this port will be used on all IP addresses associated 
>> with the server.
>
> 1. The other way,
> "this IP address will be used on all ports" ?
Ok I changed.
I have to admit I was lazy :-) and simply copy the documentation from
http://tomcat.apache.org/tomcat-7.0-doc/config/http.html
>
> 2. You code sets the address on HTTP connector. What about HTTPS and AJP ones?
Good catch fixed.
>
>> +     * @since 2.2
>> +     */
>> +    @Parameter( property = "maven.tomcat.address")
>> +    private String address;
>> +
>> +    /**
>>       * The AJP port to run the Tomcat server on.
>>       * By default it's 0 this means won't be started.
>>       * The ajp connector will be started only for value > 0.
>> @@ -909,6 +919,13 @@ public abstract class AbstractRunMojo
>>                  }
>>                  httpConnector.setURIEncoding( uriEncoding );
>>                  httpConnector.setUseBodyEncodingForURI( 
>> this.useBodyEncodingForURI );
>> +
>> +                if ( address !=null)
>> +                {
>> +                    //ici
>> +                    httpConnector.setAttribute( "address", address );
>> +                }
>
> 3. (Specific to Tomcat 6 part of this commit): It would be a bit
> easier to pass the address to Embedded.createConnector(String, int,
> String) as called few lines above. There is a variant of that method
> that accepts String as the address argument
>
>> +
>>                  container.addConnector( httpConnector );
>>
>>                  // create https connector
>>
>> Modified: 
>> tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java
>> URL: 
>> http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java?rev=1538502&r1=1538501&r2=1538502&view=diff
>> ==============================================================================
>> --- 
>> tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java
>>  (original)
>> +++ 
>> tomcat/maven-plugin/trunk/tomcat7-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java
>>  Mon Nov  4 04:12:13 2013
>>...
>
> The same for Tomcat 7.
>
> Best regards,
> Konstantin Kolinko
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>



-- 
Olivier Lamy
Ecetera: http://ecetera.com.au
http://twitter.com/olamy | http://linkedin.com/in/olamy

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to