Merge pull request #212 from mamantoha/content_length-i64

fix `Response#content_length` to be Int64
mamantoha · May 9, 2023 · 6999420 · 6999420
2 parents d270d71 + 0eba1a5
commit 6999420
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/samples/progress.cr b/samples/progress.cr
@@ -1,11 +1,11 @@
 require "../src/crest"
 
-url = "https://github.com/crystal-lang/crystal/archive/1.8.1.zip"
+url = "https://releases.ubuntu.com/23.04/ubuntu-23.04-desktop-amd64.iso"
 buffer_size = 4096
 downloaded_size = 0
 
 Crest.get(url) do |response|
-  output_file = response.filename || "crystal.zip"
+  output_file = response.filename || "ubuntu-23.04-desktop-amd64.iso"
   content_length = response.content_length
 
   File.open(output_file, "w") do |file|

diff --git a/src/crest/response.cr b/src/crest/response.cr
@@ -64,8 +64,8 @@ module Crest
     end
 
     # Size of the message body in bytes taken from "Content-Length" header
-    def content_length : Int32
-      http_client_res.headers["Content-Length"].to_i
+    def content_length : Int64
+      http_client_res.headers["Content-Length"].to_i64
     end
 
     def invalid?