Merge pull request #341 from PerimeterX/dev

GQL fix
PerimeterX · Dec 26, 2024 · 4d7b187 · 4d7b187
2 parents 2086061 + b173de2
commit 4d7b187
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/px/pxnginx.lua b/lib/px/pxnginx.lua
@@ -63,6 +63,7 @@ function M.application(px_configuration_table)
     local reverse_prefix_appid = string.sub(px_config.px_appId, 3, string.len(px_config.px_appId))
     local reverse_prefix = px_config.first_party_prefix ~= nil and px_config.first_party_prefix .. '/' .. reverse_prefix_appid or reverse_prefix_appid
     local lower_request_url = string.lower(ngx.var.request_uri)
+    local lower_request_path = string.lower(ngx.var.uri)
 
     -- Internal wrapper function, will check if uri match first party route and forward the request if uri was matched
     local function is_first_party_request(reverse_prefix, lower_request_url)
@@ -301,7 +302,7 @@ function M.application(px_configuration_table)
         ngx.ctx.pxvid = pxvid
     end
 
-    local graphql = px_graphql.extract(lower_request_url)
+    local graphql = px_graphql.extract(lower_request_path)
     if graphql then
         details["graphql_operation_name"] = graphql["operationName"]
         details["graphql_operation_type"] = graphql["operationType"]

diff --git a/lib/px/utils/config_builder.lua b/lib/px/utils/config_builder.lua
@@ -68,7 +68,7 @@ PX_DEFAULT_CONFIGURATIONS["postpone_page_requested"] = { false, "boolean"}
 PX_DEFAULT_CONFIGURATIONS["hypesale_host"] = { "https://captcha.px-cdn.net", "string"}
 PX_DEFAULT_CONFIGURATIONS["px_sensitive_graphql_operation_types"] = { {}, "table"}
 PX_DEFAULT_CONFIGURATIONS["px_sensitive_graphql_operation_names"] = { {}, "table"}
-PX_DEFAULT_CONFIGURATIONS["px_graphql_routes"] = { {'/graphql'}, "table"}
+PX_DEFAULT_CONFIGURATIONS["px_graphql_routes"] = { {}, "table"}
 PX_DEFAULT_CONFIGURATIONS["px_enable_login_creds_extraction"] = { false, "boolean"}
 PX_DEFAULT_CONFIGURATIONS["px_login_creds_settings_filename"] = { nil, "string"}
 PX_DEFAULT_CONFIGURATIONS["px_login_creds_settings"] = { nil, "table"}
@@ -131,7 +131,7 @@ function _M.load(px_config)
     end
 
     -- set default GraphQL route
-    if not px_config.px_graphql_routes then
+    if next(px_config.px_graphql_routes) == nil then
         table.insert(px_config.px_graphql_routes, px_constants.GRAPHQL_PATH)
     end