Hi Alex,

You are correct, I missed this point. Thanks for looking into it and
providing a patch.

Best regards,
Jerome  

> -----Message d'origine-----
> De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] De la 
> part de Alex Milowski
> Envoyé : dimanche 8 juillet 2007 17:22
> À : [email protected]
> Objet : Re: Server Interface Binding
> 
> On 7/7/07, Jerome Louvel <[EMAIL PROTECTED]> wrote:
> >
> > Alex,
> >
> > It looks fine to me at the Restlet level... In Component, 
> the code looks ok:
> >
> >     public Server add(Protocol protocol, int port) {
> >         Server result = new Server(getContext(), protocol, 
> null, port,
> >                 getTarget());
> >         add(result);
> >         return result;
> >     }
> 
> The port isn't the problem.  It is the binding address.
> 
> Also, you must mean ServerList.java, right ?
> 
> I see:
> 
>     public Server add(Protocol protocol, String address, int port) {
>         Server result = new Server(getContext(), protocol, 
> address, port,
>                 getTarget());
>         add(result);
>         return result;
>     }
> 
> 
> Which looks OK.
> 
> >
> > At the Simple HTTP connector level, the code is fine too:
> >
> >     @Override
> >     public void start() throws Exception {
> >         setSocket(new ServerSocket(getServer().getPort()));
> 
> The above is the problem.  If the address is specified, an 
> InetAddress instance
> needs to be specified on the constructor.  The same needs to 
> be done for https.
> 
> I think I can patch the code and see if I can get it to work.
> 
> --Alex Milowski

Reply via email to