• 10 dec 2017: forum version update. In case of issues use this topic.
  • 30 nov 2017: pilight moved servers. In case of issues use this topic.
Hello There, Guest! Login Register


Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
http library doesn't properly handle big response
#31
Do you prefer one PR for all actions, or separate ones for each one of them?
 
Reply
#32
All in one is fine. If we at least talking about the actions with the ordering checks removed.
 
Reply
#33
When testing the actions after removing order checking, I noticed that the pushover- and pushbullet actions aren't being loaded at startup because the minimal required version is 8.1.2. The latest current staging version is 8.1.1.

That was the effect of a last minute change I made after I saw your version of the http action you published on the wiki.
Stupid of me that I did not test again after making that change. Blush

Ofc. it is easy to fix, but do you want me to make PR's for this fix, now that these actions have been published in staging?
 
Reply
#34
Testing the dim action without order checking, I found that a check for the validity of the argument of the "TO" option (dimlevel) is missing. The dimlevel must be a number. If a string is entered instead of a number, an error is reported now that doesn't say what really is wrong and is unclear for the user:

Code:
ERROR: /usr/local/lib/pilight/actions/dim.lua:97: bad argument #1 to 'hasDimlevel' (number expected, got nil)

I therefore am suggesting to add an expilicit check for the validity of the TO option.

Do you you want a separate PR for this?
 
Reply
#35
No, i will fix it myself, because it maybe requires an additional unit test.
 
Reply
#36
OK, then I will wait with my PR for the removed order checking until you have pused the fix.
 
Reply
#37
Do we consider the latest staging code ready for stable?
 
Reply
#38
I cannot confirm that now, because I haven't been able to test with my full config yet. All I can say is that all issues I have seen so far have been fixed . I hope I can test later this week.
 
Reply
#39
I started testing staging with my full config. Until now, I found one new issue: SCREEN devices cannot be swithed because the SCREEN device type is not being handled in lua/config.c.

An issue I mentioned before is that the 3 second timeout is too short in many cases.

I hope to do more testing the days to come.
 
Reply
#40
I pushed the screen lua lib type.
 
Reply
  


Possibly Related Threads...
Thread Author Replies Views Last Post
  ][solved]Segfault when retrieving big chunked http message Niek 21 5,962 11-29-2018, 03:17 PM
Last Post: curlymo
  http code 301 causes segfault Niek 3 3,355 08-14-2018, 06:57 PM
Last Post: curlymo
  no callback on some http requests Niek 20 6,511 07-28-2018, 10:22 AM
Last Post: Niek

Forum Jump:


Browsing: 1 Guest(s)