Skip to content

Commit

Permalink
update ping node readme
Browse files Browse the repository at this point in the history
(how to run in snap)
  • Loading branch information
Dave Conway-Jones committed Dec 7, 2018
1 parent 89d7853 commit d0f36dc
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
4 changes: 3 additions & 1 deletion io/ping/88-ping.html
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,9 @@
<p>Pings a machine and returns the trip time in mS as <code>msg.payload</code>.</p>
<p>Returns <b>false</b> if no response received within 5 seconds, or if the host is unresolveable.</p>
<p>Default ping is every 20 seconds but can be configured.</p>
<p><code>msg.topic</code> contains the target host ip.
<p><code>msg.topic</code> contains the target host ip.</p>
<p>Note: if running inside Ubuntu Snap you will need to manually start the network-observe interface.
`snap connect node-red:network-observe`</p>
</script>

<script type="text/javascript">
Expand Down
5 changes: 4 additions & 1 deletion io/ping/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,12 @@ Run the following command in your Node-RED user directory - typically `~/.node-r

npm install node-red-node-ping


**Gotchas**

1 This won't run on Ubunti Snap as the strict container does not allow spawning of external commands (like ping).
1 Ubuntu Snap containers are strict and do not like giving external commands (like ping) external access. To allow ping to work you must manually add the network-observe interface

sudo snap connect node-red:network-observe

2 On some versions on Raspbian (Raspberry Pi) `ping` seems to be a root only command.
The fix is to allow it as follows
Expand Down
2 changes: 1 addition & 1 deletion io/ping/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name" : "node-red-node-ping",
"version" : "0.0.16",
"version" : "0.0.17",
"description" : "A Node-RED node to ping a remote server, for use as a keep-alive check.",
"dependencies" : {
},
Expand Down

0 comments on commit d0f36dc

Please sign in to comment.