Skip to content

Add option to pass through compression #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
axsuul merged 3 commits into from
Feb 8, 2022
Merged

Add option to pass through compression #27

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c9336a2
Tweak Rack request header initialization
kylewlacy Jan 29, 2018
5662cc2
Add option to pass through compressed responses
kylewlacy Jan 29, 2018
46eaa86
Replace `:compression` option with `:reset_accept_encoding`
kylewlacy Apr 19, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 15 additions & 16 deletions lib/reverse_proxy/client.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,26 +38,25 @@ def initialize(url)

def request(env, options = {}, &block)
options.reverse_merge!(
headers: {},
http: {},
path: nil,
username: nil,
password: nil,
verify_ssl: true
headers: {},
http: {},
path: nil,
username: nil,
password: nil,
verify_ssl: true,
reset_accept_encoding: false
)

source_request = Rack::Request.new(env)

# We can pass in a custom path
uri = Addressable::URI.parse("#{url}#{options[:path] || env['ORIGINAL_FULLPATH']}")

# Initialize request
target_request = Net::HTTP.const_get(source_request.request_method.capitalize).new(uri.request_uri)

# Setup headers
# Define headers
target_request_headers = extract_http_request_headers(source_request.env).merge(options[:headers])

target_request.initialize_http_header(target_request_headers)
# Initialize request
target_request = Net::HTTP.const_get(source_request.request_method.capitalize).new(uri.request_uri, target_request_headers)

# Basic auth
target_request.basic_auth(options[:username], options[:password]) if options[:username] and options[:password]
Expand All @@ -75,11 +74,11 @@ def request(env, options = {}, &block)
# Hold the response here
target_response = nil

# Don't encode response/support compression which was
# causing content length not match the actual content
# length of the response which ended up causing issues
# within Varnish (503)
target_request['Accept-Encoding'] = nil
if options[:reset_accept_encoding]
# Clear the "Accept-Encoding" header (which will
# disable compression or other server-side encodings)
target_request['Accept-Encoding'] = nil
end

http_options = {}
http_options[:use_ssl] = (uri.scheme == "https")
Expand Down