nic-6443 commented on code in PR #13386:
URL: https://github.com/apache/apisix/pull/13386#discussion_r3263265286
##########
apisix/cli/config.lua:
##########
@@ -29,6 +29,7 @@ local _M = {
enable_server_tokens = true,
extra_lua_path = "",
extra_lua_cpath = "",
+ request_body_json_lib = "qjson",
Review Comment:
Keeping qjson as the default is intentional for this change. ff40b98 reduces
the upgrade failure mode by falling back to cjson if qjson cannot be loaded,
but the default remains qjson as requested for this feature.
##########
apisix/plugins/ai-transport/http.lua:
##########
@@ -19,6 +19,7 @@
-- Provides HTTP client lifecycle management for AI provider requests.
local core = require("apisix.core")
Review Comment:
Keeping the narrower scope intentionally. This selector is for parsed
request bodies and AI upstream request body serialization; event-stream/SSE
internals are protocol framing code and should continue to use core.json unless
they become part of the same request body path.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
