Fix.

Author: ioquatix

lib/protocol/http/accept_encoding.rb

@@ -21,6 +21,7 @@ class AcceptEncoding < Middleware
 			# The default wrappers to use for decoding content.
 			DEFAULT_WRAPPERS = {
 				"gzip" => Body::Inflate.method(:for),
+				"identity" => ->(body) { body }, # Identity means no encoding
 
 				# There is no point including this:
 				# 'identity' => ->(body){body},
@@ -46,15 +47,32 @@ def call(request)
 
 				response = super
 
-				if body = response.body and !body.empty? and content_encoding = response.headers.delete(CONTENT_ENCODING)
-					# We want to unwrap all encodings
-					content_encoding.reverse_each do |name|
-						if wrapper = @wrappers[name]
-							body = wrapper.call(body)
+				if body = response.body and !body.empty?
+					if content_encoding = response.headers[CONTENT_ENCODING]
+						# Process encodings from the end (last applied first)
+						# Remove encodings as we successfully decode them
+						while name = content_encoding.last
+							# Look up wrapper with case-insensitive matching
+							wrapper = @wrappers[name] || @wrappers[name.downcase]
+							
+							if wrapper
+								body = wrapper.call(body)
+								# Remove the encoding we just processed:
+								content_encoding.pop
+							else
+								# Unknown encoding - stop processing here:
+								break
+							end
+						end
+						
+						# Update the response body:
+						response.body = body
+						
+						# Remove the content-encoding header if we decoded all encodings:
+						if content_encoding.empty?
+							response.headers.delete(CONTENT_ENCODING)
 						end
 					end
-					
-					response.body = body
 				end
 
 				return response

test/protocol/http/accept_encoding.rb

@@ -117,9 +117,6 @@
 			# Mock upstream server that returns brotli-encoded content
 			upstream_delegate = ->(request) {
 				# Simulate a server responding with brotli encoding
-				# when the request has accept-encoding: gzip
-				expect(request.headers["accept-encoding"]).to be == ["gzip"]
-				
 				Protocol::HTTP::Response[200, 
 					Protocol::HTTP::Headers[
 						"content-type" => "text/html",