You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@vcl.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2018/06/06 19:55:00 UTC

[jira] [Commented] (VCL-1089) Change ping module to use Net::Ping::External

    [ https://issues.apache.org/jira/browse/VCL-1089?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16503833#comment-16503833 ] 

ASF subversion and git services commented on VCL-1089:
------------------------------------------------------

Commit a087631acf134dc06788cb855137f3d172831668 in vcl's branch refs/heads/develop from [~jfthomps]
[ https://git-wip-us.apache.org/repos/asf?p=vcl.git;h=a087631 ]

VCL-1089 - Change ping module to use Net::Ping::External

-applied patch submitted by Mike Jennings that changes utils.pm
_pingnode function to use Net::Ping::External rather than Net::Ping so
that vcld doesn't require root level rights
-moved "use Net::Ping::External qw(ping);" line to top of file with other
calls to "use"
-updated description of _pingnode function


> Change ping module to use Net::Ping::External
> ---------------------------------------------
>
>                 Key: VCL-1089
>                 URL: https://issues.apache.org/jira/browse/VCL-1089
>             Project: VCL
>          Issue Type: Improvement
>          Components: vcld (backend)
>    Affects Versions: 2.5
>            Reporter: Mike Jennings
>            Priority: Major
>             Fix For: 2.5.1
>
>         Attachments: vcl_1089.patch
>
>
> Currently VCL uses the Net::Ping->new("icmp"); module to ping computers.  This module can currently only be run as the root user.
> I would suggest that we change the ping module from 
> my $p = Net::Ping->new("icmp");
> my $result = $p->ping($remote_connection_target, 1);
> $p->close();
> to
> use Net::Ping::External qw(ping);
> my $result = ping(host => $remote_connection_target, timeout => 1);
>  
> This will help allow vcl to be run as a user other than root.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)