From 3a6e833eaec1e187aec1f2ac438cc8951a23d661 Mon Sep 17 00:00:00 2001 From: Joshua Marantz Date: Thu, 22 Oct 2015 14:19:00 -0400 Subject: [PATCH] Don't call RewriteDriver::SetUserAgent on the UA in the request-headers. RewriteDriver::SetRequestHeaders already pulls the UA from the request-headers. --- src/ngx_pagespeed.cc | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/ngx_pagespeed.cc b/src/ngx_pagespeed.cc index 7e917f3ae..4ef079282 100644 --- a/src/ngx_pagespeed.cc +++ b/src/ngx_pagespeed.cc @@ -1903,11 +1903,6 @@ ngx_int_t ps_resource_handler(ngx_http_request_t* r, custom_options.release(), ctx->base_fetch->request_context()); } - StringPiece user_agent = ctx->base_fetch->request_headers()->Lookup1( - HttpAttributes::kUserAgent); - if (!user_agent.empty()) { - driver->SetUserAgent(user_agent); - } driver->SetRequestHeaders(*ctx->base_fetch->request_headers()); driver->set_pagespeed_query_params(pagespeed_query_params); driver->set_pagespeed_option_cookies(pagespeed_option_cookies); @@ -1948,13 +1943,7 @@ ngx_int_t ps_resource_handler(ngx_http_request_t* r, custom_options.release(), ctx->base_fetch->request_context()); } - StringPiece user_agent = ctx->base_fetch->request_headers()->Lookup1( - HttpAttributes::kUserAgent); - if (!user_agent.empty()) { - driver->SetUserAgent(user_agent); - } driver->SetRequestHeaders(*ctx->base_fetch->request_headers()); - ctx->driver = driver; cfg_s->server_context->message_handler()->Message(