[5/6] alfred-monitoring-proxy: Switch to alfred2

Submitted by Adrian Schmutzler on Jan. 27, 2019, 4:38 p.m.

Details

Message ID 20190127163823.23382-6-freifunk@adrianschmutzler.de
State Accepted
Headers show

Commit Message

Adrian Schmutzler Jan. 27, 2019, 4:38 p.m.
By using the alfred2 mechanism of the Monitoring, we get rid
of the embracing {"64":...} key-value construct.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 src/packages/fff/fff-alfred-monitoring-proxy/Makefile                 | 2 +-
 .../files/usr/sbin/alfred-monitoring-proxy                            | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
index 7ebc2ee0..4370bd9c 100644
--- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
+++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
@@ -2,7 +2,7 @@  include $(TOPDIR)/rules.mk
 
 PKG_NAME:=fff-alfred-monitoring-proxy
 PKG_VERSION:=0.0.1
-PKG_RELEASE:=1
+PKG_RELEASE:=2
 
 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
 
diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy b/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy
index c35e7f97..b91187f6 100755
--- a/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy
+++ b/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy
@@ -1,7 +1,7 @@ 
 #!/bin/ash
 
-api_url="https://monitoring.freifunk-franken.de/api/alfred"
+api_url="https://monitoring.freifunk-franken.de/api/alfred2"
 fetch_id="64"
 
-(echo -n "{\"$fetch_id\": "; alfred-json -r "$fetch_id"; echo -n "}") | \
+/bin/alfred-json -r "$fetch_id" | \
     curl -k -v -H "Content-type: application/json; charset=UTF-8" -X POST --data "@-" $api_url

Comments

Christian Dresel Jan. 27, 2019, 4:58 p.m.
Hi

ich weiß zwar jetzt nicht genau, warum du da 2 Patches draus gemacht
hast, denn ich bin zuerst an diesen hier verzweifelt und wollte schon
aufgeben es zu verstehen, bis ich dann bei 6/6 gelandet bin und bemerkt
habe, das ich es gemeinsam mit diesen dann verstehe ;) Denn genauso wie
es dann nach 6/6 ist verwende ich es bereits auf fff-nue2-gw3 und es tut
sehr gut :)

Reviewed-by: Christian Dresel <fff@chrisi01.de>

Gruß

Christian

On 27.01.19 17:38, Adrian Schmutzler wrote:
> By using the alfred2 mechanism of the Monitoring, we get rid
> of the embracing {"64":...} key-value construct.
>
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  src/packages/fff/fff-alfred-monitoring-proxy/Makefile                 | 2 +-
>  .../files/usr/sbin/alfred-monitoring-proxy                            | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
> index 7ebc2ee0..4370bd9c 100644
> --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
> +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
> @@ -2,7 +2,7 @@ include $(TOPDIR)/rules.mk
>  
>  PKG_NAME:=fff-alfred-monitoring-proxy
>  PKG_VERSION:=0.0.1
> -PKG_RELEASE:=1
> +PKG_RELEASE:=2
>  
>  PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
>  
> diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy b/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy
> index c35e7f97..b91187f6 100755
> --- a/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy
> +++ b/src/packages/fff/fff-alfred-monitoring-proxy/files/usr/sbin/alfred-monitoring-proxy
> @@ -1,7 +1,7 @@
>  #!/bin/ash
>  
> -api_url="https://monitoring.freifunk-franken.de/api/alfred"
> +api_url="https://monitoring.freifunk-franken.de/api/alfred2"
>  fetch_id="64"
>  
> -(echo -n "{\"$fetch_id\": "; alfred-json -r "$fetch_id"; echo -n "}") | \
> +/bin/alfred-json -r "$fetch_id" | \
>      curl -k -v -H "Content-type: application/json; charset=UTF-8" -X POST --data "@-" $api_url
Fabian Blaese Jan. 27, 2019, 8:27 p.m.
Hallo,

On 27.01.19 17:58, Christian Dresel wrote:
> Hi
> 
> ich weiß zwar jetzt nicht genau, warum du da 2 Patches draus gemacht
> hast, denn ich bin zuerst an diesen hier verzweifelt und wollte schon
> aufgeben es zu verstehen, bis ich dann bei 6/6 gelandet bin und bemerkt
> habe, das ich es gemeinsam mit diesen dann verstehe ;) Denn genauso wie
> es dann nach 6/6 ist verwende ich es bereits auf fff-nue2-gw3 und es tut
> sehr gut :)
das verstehe ich jetzt nicht ganz. Das eine hat mit dem andern erstmal nichts zu tun, beide Patches lassen sich einzeln Applien, ohne dass etwas bricht..

Ungetestet, sieht aber gut aus.

Reviewed-by: Fabian Bläse <fabian@blaese.de>

Fabian