Skip to content

Commit 1fe8286

Browse files
authored
Adapt to Piaf's latest interface (#54)
1 parent 4b22f63 commit 1fe8286

File tree

4 files changed

+23
-15
lines changed

4 files changed

+23
-15
lines changed

lib/client.ml

+16-9
Original file line numberDiff line numberDiff line change
@@ -135,8 +135,7 @@ let event_response client request_id output =
135135
request_id
136136
in
137137
make_runtime_post_request client path output >>= function
138-
| Ok response ->
139-
let status = Response.status response in
138+
| Ok { Response.status; _ } ->
140139
if not (Status.is_successful status) then
141140
let error =
142141
Errors.make_api_error
@@ -179,8 +178,7 @@ let event_error client request_id err =
179178
request_id
180179
in
181180
make_runtime_error_request client path err >>= function
182-
| Ok response ->
183-
let status = Response.status response in
181+
| Ok { Response.status; _ } ->
184182
if not (Status.is_successful status) then
185183
let error =
186184
Errors.make_api_error
@@ -281,9 +279,7 @@ let next_event client =
281279
Logs_lwt.info (fun m -> m "Polling for next event. Path: %s\n" path)
282280
>>= fun () ->
283281
Client.get client path >>= function
284-
| Ok response ->
285-
let headers = Response.headers response in
286-
let status = Response.status response in
282+
| Ok { Response.status; headers; body; _ } ->
287283
let code = Status.to_code status in
288284
if Status.is_client_error status then
289285
Logs_lwt.err (fun m ->
@@ -316,8 +312,19 @@ let next_event client =
316312
m "Failed to get event context: %s\n" (Errors.message err))
317313
>>= fun () -> Lwt_result.fail err
318314
| Ok ctx ->
319-
Body.to_string (Response.body response) >>= fun body_str ->
320-
Lwt_result.return (body_str, ctx))
315+
Body.to_string body >>= ( function
316+
| Ok body_str ->
317+
Lwt_result.return (body_str, ctx)
318+
| Error e ->
319+
let err =
320+
Errors.make_api_error
321+
~recoverable:false
322+
(Format.asprintf
323+
"Server error when polling for new events: %a"
324+
Piaf.Error.pp_hum
325+
e)
326+
in
327+
Lwt_result.fail err ))
321328
| Error _ ->
322329
let err =
323330
Errors.make_api_error

lib/runtime.ml

+3-2
Original file line numberDiff line numberDiff line change
@@ -147,8 +147,9 @@ struct
147147
make ~max_retries:3 ~settings:function_config ~lift ~handler client
148148
in
149149
start runtime
150-
| Error msg ->
151-
failwith (Format.asprintf "Could not start HTTP client: %s" msg)
150+
| Error e ->
151+
failwith
152+
(Format.asprintf "Could not start HTTP client: %a" Piaf.Error.pp_hum e)
152153

153154
let start_lambda ~lift handler =
154155
match Config.get_runtime_api_endpoint () with

nix/default.nix

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ in
88
with ocamlPackages;
99
let
1010
build-lambda-runtime = args: buildDunePackage ({
11+
useDune2=true;
1112
version = "0.1.0-dev";
1213
src = lib.gitignoreSource ./..;
1314
} // args);

nix/sources.nix

+3-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
11
{ ocamlVersion ? "4_10" }:
22

33
let
4-
overlays = builtins.fetchTarball {
5-
url = https://github.com/anmonteiro/nix-overlays/archive/9ebb27af.tar.gz;
6-
sha256 = "0qlq4g96scz1lqsiyprkggjnzhxga07csqjc75ax1j1kk1ich34i";
7-
};
4+
overlays =
5+
builtins.fetchTarball
6+
https://github.com/anmonteiro/nix-overlays/archive/3fb1555.tar.gz;
87

98
in
109

0 commit comments

Comments
 (0)