Re: [LEDE-DEV] [PATCH 3/3] Add debug output for service_timeout

2017-10-02 Thread Philipp Meier
On 09/28/2017 09:35 PM, Philip Prindeville wrote: Inline Sent from my iPhone On Sep 28, 2017, at 1:09 AM, Philipp Meier <philipp.me...@neratec.com> wrote: Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- service.c | 4 +++- 1 file changed, 3 insertions(+), 1 delet

Re: [LEDE-DEV] [PATCH] umdns: Remove incorrect comma in http service json config

2017-09-28 Thread Philipp Meier
On 26/09/17 10:55, Philipp Meier wrote: Remove trailing comma in http service json configuration file Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- json/http.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/json/http.json b/json/http.json index c

Re: [LEDE-DEV] [PATCH] umdns: Remove incorrect comma in http service json config

2017-09-28 Thread Philipp Meier
On 26/09/17 10:55, Philipp Meier wrote: Remove trailing comma in http service json configuration file Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- json/http.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/json/http.json b/json/http.json index c

[LEDE-DEV] [PATCH 3/3] Add debug output for service_timeout

2017-09-28 Thread Philipp Meier
Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- service.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/service.c b/service.c index 0a9e25d..97b6f91 100644 --- a/service.c +++ b/service.c @@ -121,8 +121,10 @@ service_timeout(struct serv

[LEDE-DEV] [PATCH 1/3] Remove ttl==255 restriction for queries

2017-09-28 Thread Philipp Meier
Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/interface.c b/interface.c index 3904c89..7f814d2 100644 --- a/interface.c +++ b/interface.c @@ -233,9 +233,6 @@ read_socket4(struct uloop_fd *u, unsigned int

[LEDE-DEV] [PATCH 2/3] Remove incorrect comma in http service json config

2017-09-28 Thread Philipp Meier
Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- json/http.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/json/http.json b/json/http.json index cf60532..2a7b50a 100644 --- a/json/http.json +++ b/json/http.json @@ -1,3 +1,3 @@ { - "http_80&q

[LEDE-DEV] umdns: query response only every 60s

2017-09-26 Thread Philipp Meier
Hi When using umdns I experience the follow not expected behavior: There is no mDNS response sent for a service where an announcement or a response has been sent in the last 60 seconds (see service.c function service_timeout). 1. I started the LEDE device 2. Then I started a PC application

Re: [LEDE-DEV] umdns - TTL restricted to 255

2017-09-26 Thread Philipp Meier
Crispin wrote: in that case its a bug and the code should be changed to only have that restriction on queries     John On 25/09/17 17:21, Philipp Meier wrote: My question is about query (not response). LEDE is ignoring query when TTL != 255. Philipp On 09/25/2017 05:16 PM, John Crispin

[LEDE-DEV] [PATCH] umdns: Add debug output for service_timeout

2017-09-26 Thread Philipp Meier
Debug output for case where no response is sent out due to service_timeout. This would have saved me some ours in finding out why an mDNS query did not get any response. Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- service.c | 4 +++- 1 file changed, 3 insertions

[LEDE-DEV] [PATCH] umdns: Remove incorrect comma in http service json config

2017-09-26 Thread Philipp Meier
Remove trailing comma in http service json configuration file Signed-off-by: Philipp Meier <philipp.me...@neratec.com> --- json/http.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/json/http.json b/json/http.json index cf60532..2a7b50a 100644 --- a/json/http.json

[LEDE-DEV] [PATCH] Remove ttl==255 restriction for queries

2017-09-26 Thread Philipp Meier
This commit removes ttl==255 restriction for queries. Newer mDNS implementations use ttl != 255 when sending queries. Signed-off-by: Philipp Meier <philipp.me...@neratec.com> ---  interface.c | 6 --  1 file changed, 6 deletions(-) diff --git a/interface.c b/interface.c index 3

Re: [LEDE-DEV] umdns - TTL restricted to 255

2017-09-25 Thread Philipp Meier
.  These older queriers discard    all packets with TTLs other than 255.     John On 25/09/17 16:44, Philipp Meier wrote: Hi, When using umdns I was wondering why my mDNS query did not get any answer. I found the following reason: My mDNS query has TTL field set to 1. But LEDE umdns package expects

[LEDE-DEV] umdns - TTL restricted to 255

2017-09-25 Thread Philipp Meier
Hi, When using umdns I was wondering why my mDNS query did not get any answer. I found the following reason: My mDNS query has TTL field set to 1. But LEDE umdns package expects a TTL value of 255 (see interface.c function read_socket4). According