[haiku-commits] Re: haiku: hrev50368 - src/kits/network/libnetapi headers/os/net

  • From: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 21 Jun 2016 23:23:57 +0200

On Tue, Jun 21, 2016 at 06:43:23PM +0100, Hamish Morrison wrote:

--- a/headers/os/net/NetworkAddress.h
+++ b/headers/os/net/NetworkAddress.h
@@ -149,6 +149,7 @@ private:
 private:
                        sockaddr_storage        fAddress;
                        status_t                        fStatus;
+                       BString                         fHostName;
 };

On the subject of the fix itself, adding the host name string to
BNetworkAddress just to cover this special case seems wrong to me.
BNetworkAddress should be agnostic about the host name it was resolved
from (indeed it might not even have a host name at all).

I would instead simply add a method to SecureSocket to allow setting
the hostname used for the server name indication.

It can be useful in other cases as well. For example, an HTTP request
usually needs to know which hostname is used as it specifies it in the
request header. There may be similar uses in other protocols.

It depends if we want to consider BNetworkAddress as IPv4/6 centric, or
something a bit higher level and more friendly to upper layer protocols.

-- 
Adrien.

Other related posts: