Skip to content

Commit

Permalink
Remove full_path from request
Browse files Browse the repository at this point in the history
	modified:   spec/awscr-signer/v4/request_spec.cr
	modified:   src/awscr-signer/v4/request.cr
  • Loading branch information
jrester committed Apr 29, 2020
1 parent acd67d6 commit facd698
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
9 changes: 0 additions & 9 deletions spec/awscr-signer/v4/request_spec.cr
Original file line number Diff line number Diff line change
Expand Up @@ -44,15 +44,6 @@ module Awscr
end
end

describe "full_path" do
it "returns the full url incl query string" do
request = Request.new("GET", "http://google.com", "")
request.query.add("blah", "1")

request.full_path.should eq("http://google.com?blah=1")
end
end

describe "to_s" do
it "returns a valid string" do
body = ""
Expand Down
4 changes: 0 additions & 4 deletions src/awscr-signer/v4/request.cr
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,6 @@ module Awscr
@headers["Host"]?.try(&.value)
end

def full_path
"#{@path}?#{query}"
end

# Returns the request as a String.
def to_s(io : IO)
io << [
Expand Down

0 comments on commit facd698

Please sign in to comment.