Isaline L.
07/15/2021, 12:34 PMs
Stream
class (instead of HttpStream
directly) and use that.
In general, it might be helpful to consider if the Stream
class is a better fit than HttpStream
, Iād be curious to know how it ends up!Isaline L.
07/15/2021, 2:50 PM_create_prepared_request
from HttpStream to add the "json" (which is actually xml then) into args["data"] instead of args["json"] and it works.Isaline L.
07/15/2021, 2:51 PMIsaline L.
07/15/2021, 2:52 PMIsaline L.
07/15/2021, 2:56 PMjson
parameter of request, but the data
parameter. So what I would do is offering another method next to request_body_json
, like request_body_data
. People are free to implement whichever they need, and you can call both of them next to each other in read_records
.Isaline L.
07/15/2021, 2:56 PMIsaline L.
07/15/2021, 2:57 PMdata
and json
are provided to requests, but that should be easy to find out. ^^s
Isaline L.
07/15/2021, 3:50 PMIsaline L.
07/15/2021, 3:50 PMs