robur/builder-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Check if-not-modified

Browse source 
We are still reading the whole file from disk even when it's not needed.
This commit is contained in:
Reynir Björnsson 2021-06-04 12:52:35 +02:00
parent 1ce4c11c05
commit 847e0cffe4
1 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/builder_web.ml
View file

@ -183,6 +183,7 @@ let add_routes datadir =
let _job_name = Dream.param "job" req let _job_name = Dream.param "job" req
and build = Dream.param "build" req and build = Dream.param "build" req
and filepath = Dream.path req |> String.concat "/" in and filepath = Dream.path req |> String.concat "/" in
let if_none_match = Dream.header "if-none-match" req in
(* XXX: We don't check safety of [file]. This should be fine however since (* XXX: We don't check safety of [file]. This should be fine however since
* we don't use [file] for the filesystem but is instead used as a key for * we don't use [file] for the filesystem but is instead used as a key for
* lookup in the data table of the 'full' file. *) * lookup in the data table of the 'full' file. *)
@ -200,9 +201,14 @@ let add_routes datadir =
Log.warn (fun m -> m "Error getting build artifact: %a" pp_error e); Log.warn (fun m -> m "Error getting build artifact: %a" pp_error e);
Dream.respond ~status:`Internal_Server_Error "Error getting build artifact" Dream.respond ~status:`Internal_Server_Error "Error getting build artifact"
| Ok (data, digest) -> | Ok (data, digest) ->
let etag = Base64.encode_string (Cstruct.to_string digest) in
match if_none_match with
| Some etag' when etag = etag' ->
Dream.empty `Not_Modified
| _ ->
let headers = [ let headers = [
"Content-Type", mime_lookup filepath; "Content-Type", mime_lookup filepath;
"ETag",(Base64.encode_string (Cstruct.to_string digest)); "ETag", etag;
] in ] in
Dream.respond ~headers data Dream.respond ~headers data
in in