Http Post using http.request results in bad request

Developer's Corner General Dev Discussions
1

Hello,

After days of surfing the web, reading forum topics and creating requests in scenes, I feel defeated and need some fresh idea’s.
I’m trying to send a simple HTTP POST request to a Milestone XProtect system, containing an xml body.
Simple enough I would think…
Started by creating a scene in my Vera Edge.
Added the following code:

[code]
local http = require(“socket.http”)

http.TIMEOUT = 5

local response_body = { }

local body =[[<?xml version="1.0" encoding="utf-8"?>00000000-0000-0000-0000-0000000000002017-05-02T20:48:12MyTypeMyAnalyticsEvent01TagFromXML192.168.1.32Analytics event description.Event location 1My Smart Video]]

local res, code, response_headers, status = http.request{
url = “http://192.168.1.116:9090”,
sink = ltn12.sink.table(response_body),
headers = {
[“Content-Type”] = “text/xml”,
[“Content-Length”] = tostring(body:len()),
[“Connection”] = “Keep-Alive”
},
source = ltn12.source.string(body),
method = “POST”
}

luup.task('Response: = ’ … table.concat(response_body) … ’ code = ’ … code … ’ status = ’ … status,1,‘Sample POST request with JSON data’,-1)[/code]

When triggering this scene, it runs, but I get a message: Sample POST request with JSON data : Response: = Error: Invalid data code = 400 status = HTTP/1.1 400 Bad Request

Using wireshark on the server monitoring port 9090 I see there is a partial packet, the server response with 400 and after the response there is the rest of the POST (2 segments?)

Using a custom program to send the same xml data, all works fine. (Attached are both the wireshark captures)

I found things like this: [url=http://forum.micasaverde.com/index.php?topic=30514.0]http://forum.micasaverde.com/index.php?topic=30514.0[/url]
But it is still not working correctly…

HELP :o :wink: :smiley:

I hope I provided enough information so that anyone can help me get back on track…

Thanks in advance

With kind regards,

Jan

2

Solved my own problem.

I noticed the first package contained only the “POST / HTTP/1.1\r\n” portion.
After reading and interpreting the http.lua file. I noticed the next two lines:

h:sendrequestline(nreqt.method, nreqt.uri)
h:sendheaders(nreqt.headers)

So after sending the request line, my server is already responding.
Afterwards, the complete package was received.

So I combined the two functions to a single one whitch sends the requestline and headers in a singe c:send

Added a custom property to the request, when set it will use my custom function (to not disturb other use of http.socket)

Done!

All works nicely

If someone is interested in more details, just ask 8)

3

Had exactly the same problem on RasPi. Glad to see it wasn’t just me. See attached file: lines 122 to 148.

Would be good to get this permanently fixed. The code would also be more efficient, as it would only send one packet instead of two. Not sure what the RFCs say but maybe our servers are not behaving?

Need to watch out for this as well:

http://forum.micasaverde.com/index.php?topic=13081.0

On your code - you might want to include a require of ltn12.

What’s the Milestone XProtect set up like?

4

I believe this is a server problem … but most clients send the header contents in the original message … so it is not well tested.

The server is supposed to accept headers until it gets a double CRLF .
Since the initial packet only contained one … it should have kept reading for headers …

5

I agree with RichardTSchaefer. But because I will be waiting to long for Milestone to review their server, I have to solve the problem on the Vera side… :frowning:

I had almost an identical (or complete identical) function as a-lurker, but added a switch so other users of http.lua aren’t affected by my changes. (See file attached trequest function)
oh, and originally I added the ltn12 reference, just fell off when sending the post :smiley:

The milestone setup… Using the protocol integration, I’m going to send xml events to the analytics event server (activated) to send alarms to the milestone alarm manager. This for a control room now using milestone to monitor an environment and to handle their alarms. Now we add a home automation part wich will produce some alarms in milestone.

Thanks for the info

Juvo