kakserpom / phpdaemon

Asynchronous server-side framework for network applications implemented in PHP using libevent
http://daemon.io/
GNU Lesser General Public License v3.0
1.53k stars 231 forks source link

Bugfix: skip commented ips #272

Closed x-soft-ua closed 7 years ago

x-soft-ua commented 7 years ago

При наличии комментированных хостов в /etc/hosts и использовании этих хостов получаем неверный ip со всеми вытекающими. В нашем случае - утечкой и крашем демона :-)

ssbxlan commented 7 years ago

Возможно, вообще не стоит пытаться отрезолвить имя асинхронно, а переложить это на плечи системы. Ибо происходит это быстро, а отловить все возможные варианты малореально. Кроме того, регулярки ведь сами по себе работают не весело. А в ядре линуха, подозреваю, кошерный конечный автомат.

kakserpom commented 7 years ago

Регулярка в таком виде не очень, так как # может быть не в начале строки

kakserpom commented 7 years ago

ssbxlan, ещё как стоит