ats_getbestaddrinfo is already implemented. It's in lib/ts/ink_inet.cc. It's a
bit hard to use, I need to improve it :-) but you can see examples if you look
at the commits for those issues. The issue is that you can get back both an
IPv4 and an IPv6 address. I've filed TS-1070 to remind me to c
Sure, on all of those tickets you just moved directly to getaddrinfo(), and
I still need to write ats_getbestaddrinfo(), correct?
Brian
On Sat, Mar 31, 2012 at 6:49 PM, Alan M. Carroll <
a...@network-geographics.com> wrote:
> I have fixed TS-1166, TS-1167, TS-1168, TS-1017 all of which eliminate
I have fixed TS-1166, TS-1167, TS-1168, TS-1017 all of which eliminate uses of
gethostbyname. Looks to me like only clustering, web management,
CoreAPIShared.cc in mgmt, and some test code are still problematic. Except for
clustering those look like using ats_getbestaddrinfo() would work (since
Great. Then it seems like it is time to make the move into the 21st
century.
john
On Tue, Mar 27, 2012 at 3:46 PM, Alan M. Carroll <
a...@network-geographics.com> wrote:
> I would presume that applies to the functions documented on the page, not
> getaddrinfo which is mentioned only in passing.
I would presume that applies to the functions documented on the page, not
getaddrinfo which is mentioned only in passing. This is reenforced by
getaddrinfo having its own documentation page.
Just for amusement value, here's a couple of bug reports of gethostbyaddr_r()
crashing under load -
Tue
Search for the line:
"The lookup functions above all have one in common: they are not reentrant
and therefore unusable in multi-threaded applications. Therefore provides
the GNU C library a new set of functions which can be used in this context."
Perhaps it operates differently if _POSIX_SOURCE i
Where do you see that? My browser claims only one mention of getaddrinfo on
that page
"You can also use getaddrinfo and getnameinfo to obtain this information"
which contains no mention of thread safety. The glibc documentation for
getaddinfo
http://www.gnu.org/software/gnulib/manual/html_node
FYI: According to
http://www.gnu.org/software/libc/manual/html_node/Host-Names.html
The glibc versions of getaddrinfo are still not thread safe.
john
On Mon, Mar 26, 2012 at 7:58 PM, Alan M. Carroll <
a...@network-geographics.com> wrote:
> I filed TS-1166, TS-1167, TS-1168, to track some of th
I filed TS-1166, TS-1167, TS-1168, to track some of these places. TS-990 tracks
the Cluster issue, and TS-1017 tracks it in logging. I am working on the
logging. I will try to get through these others although others are welcome to
step up :-).
Mostly reasonable, just a few things to do better:
1) Do not use the "ink_" prefix, we're moving to "ats_".
2) It's not clear to me we need to wrap getaddrinfo at all unless you can
simplify the api (e.g., ats_getbestaddrinfo).
3) Looking at the call sites which still use ink_gethostbyname, I w
On Mar 25, 2012, at 11:06 PM, Brian Geffon wrote:
> Hi all,
> In an effort to get ATS to build on OpenBSD, I noticed that we're using
> gethostbyname_r and gethostbyaddr_r (see ink_inet.cc), according to the
> manpage both of these APIs are obsolete and getaddrinfo should be used
> (which is reen
- Original Message -
>
>
> - Original Message -
> > Hi all,
> > In an effort to get ATS to build on OpenBSD, I noticed that we're
> > using
> > gethostbyname_r and gethostbyaddr_r (see ink_inet.cc), according to
> > the
> > manpage both of these APIs are obsolete and getaddrinfo s
- Original Message -
> Hi all,
> In an effort to get ATS to build on OpenBSD, I noticed that we're
> using
> gethostbyname_r and gethostbyaddr_r (see ink_inet.cc), according to
> the
> manpage both of these APIs are obsolete and getaddrinfo should be
> used
> (which is reentrant by defaul
Hi all,
In an effort to get ATS to build on OpenBSD, I noticed that we're using
gethostbyname_r and gethostbyaddr_r (see ink_inet.cc), according to the
manpage both of these APIs are obsolete and getaddrinfo should be used
(which is reentrant by default), I only noticed this because OpenBSD 5.0
doe
14 matches
Mail list logo
