JoeDog / siege

Siege is an http load tester and benchmarking utility
GNU General Public License v3.0
5.9k stars 386 forks source link

Add configure check for getaddrinfo() #232

Closed abs0 closed 8 months ago

abs0 commented 8 months ago

Extend use of getaddrinfo() from defined(GLIBC) to any system where configure can find it.

Avoids falling back to (usually non thread-safe) gethostbyname() on BSD systems, which does not end well when concurrency > 1

JoeDog commented 8 months ago

Just want you to know that I got your request. On the surface, I can't see why it won't be accepted. I just need to test it out.

On Sat, Dec 2, 2023 at 11:46 AM David Brownlee @.***> wrote:

Extend use of getaddrinfo() from defined(GLIBC) to any system where configure can find it.

Avoids falling back to (usually non thread-safe) gethostbyname() on BSD systems, which does not end well when concurrency > 1

You can view, comment on, or merge this pull request online at:

https://github.com/JoeDog/siege/pull/232 Commit Summary

File Changes

(2 files https://github.com/JoeDog/siege/pull/232/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/JoeDog/siege/pull/232, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJRHZXIDMF6XONDGVKGZN3YHNLNJAVCNFSM6AAAAABAED4OW2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZDEMJRHEYTMOA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Jeff Fulmer 1-717-799-8226 https://www.joedog.org/ He codes