Re: [monitoring-plugins/monitoring-plugins] Update .travis.yml (#1440)

Elan Ruusamäe <[email protected]> Tue, 08 Nov 2016 14:13:11 -0800
Newsgroups gmane.network.nagios.devel
Message-ID <monitoring-plugins/monitoring-plugins/pull/1440/[email protected]>
----==_mimepart_58224df760f1a_40183f88d28591301233f8
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

@sni you should understand how shell parser works. wc and grep are not ran
under sudo, they are next command in pipe chain, if *want* them to run under
sudo, you must prepend sudo to each of them

why it works is likely "ip" command does not require root access, and sudo
will just include `/sbin` in path so the program is visible. solution: use
abs path.

however i kept sudo, because depending on kernel hardening `ip` via `sudo`
may give more information than under normal user.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/monitoring-plugins/monitoring-plugins/pull/1440#issuecomment-259276088
----==_mimepart_58224df760f1a_40183f88d28591301233f8
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p><a href=3D"https://github.com/sni" class=3D"user-mention">@sni</a> you s=
hould understand how shell parser works. wc and grep are not ran under sudo=
, they are next command in pipe chain, if <em>want</em> them to run under s=
udo, you must prepend sudo to each of them</p>

<p>why it works is likely "ip" command does not require root access, and su=
do will just include <code>/sbin</code> in path so the program is visible. =
solution: use abs path.</p>

<p>however i kept sudo, because depending on kernel hardening <code>ip</cod=
e> via <code>sudo</code> may give more information than under normal user.<=
/p>

<p style=3D"font-size:small;-webkit-text-size-adjust:none;color:#666;">&mda=
sh;<br />You are receiving this because you are subscribed to this thread.<=
br />Reply to this email directly, <a href=3D"https://github.com/monitoring=
-plugins/monitoring-plugins/pull/1440#issuecomment-259276088">Reply to this=
 email on GitHub</a>, or <a href=3D"https://github.com/notifications/unsubs=
cribe-auth/AFQl2RfPw_iHF5EWfxB-drnOI_vxM-7Vks5q8PP3gaJpZM4KryeK">mute the t=
hread</a>.<img alt=3D"" height=3D"1" src=3D"https://github.com/notification=
s/beacon/AFQl2flMa1r1yb41eWqKWuhxHYAYhkASks5q8PP3gaJpZM4KryeK.gif" width=3D=
"1" /></p>
<div itemscope itemtype=3D"http://schema.org/EmailMessage">
<div itemprop=3D"action" itemscope itemtype=3D"http://schema.org/ViewAction=
">
  <link itemprop=3D"url" href=3D"https://github.com/monitoring-plugins/moni=
toring-plugins/pull/1440#issuecomment-259276088"></link>
  <meta itemprop=3D"name" content=3D"View Pull Request"></meta>
</div>
<meta itemprop=3D"description" content=3D"View this Pull Request on GitHub"=
></meta>
</div>

<script type=3D"application/json" data-scope=3D"inboxmarkup">{"api_version"=
:"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"Gi=
tHub"},"entity":{"external_key":"github/monitoring-plugins/monitoring-plugi=
ns","title":"monitoring-plugins/monitoring-plugins","subtitle":"GitHub repo=
sitory","main_image_url":"https://cloud.githubusercontent.com/assets/143418=
/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"htt=
ps://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-=
9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://git=
hub.com/monitoring-plugins/monitoring-plugins"}},"updates":{"snippets":[{"i=
con":"PERSON","message":"@glensc in #1440: @sni you should understand how s=
hell parser works. wc and grep are not ran under sudo, they are next comman=
d in pipe chain, if *want* them to run under sudo, you must prepend sudo to=
 each of them\r\n\r\nwhy it works is likely \"ip\" command does not require=
 root access, and sudo will just include `/sbin` in path so the program is =
visible. solution: use abs path.\r\n\r\nhowever i kept sudo, because depend=
ing on kernel hardening `ip` via `sudo` may give more information than unde=
r normal user."}],"action":{"name":"View Pull Request","url":"https://githu=
b.com/monitoring-plugins/monitoring-plugins/pull/1440#issuecomment-25927608=
8"}}}</script>

----==_mimepart_58224df760f1a_40183f88d28591301233f8--