fix(ai-proxy): abstract a base for ai-proxy #11991
Merged
+128
−105
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
In the previous
ai-proxy-multi
feat PR: #11986, theai-proxy-multi
module would overwrite theai-proxy
module functions.Due to this, the
ai-proxy
plugin would executeai-proxy-multi
plugin's logic. This issue was found during some manual tests. The CI could not detect because it could not run properly as test::nginx could not initialise an nginx process as it could not find a certificate specified in the following test:apisix/t/plugin/ai-proxy-multi2.t
Lines 291 to 296 in cc7441f
The CI failure log: https://github.com/apache/apisix/actions/runs/13489934366/job/37686455127#step:19:190
Checklist