moonming commented on PR #12842: URL: https://github.com/apache/apisix/pull/12842#issuecomment-4064882353
Hi @zhl11b, thank you for submitting this! I have a few concerns with the current state of this PR: 1. **Missing context**: There's no associated issue describing the problem this fixes. Could you share the specific scenario where you encountered this issue? 2. **No test coverage**: The single-line change to `ext-plugin-post-resp.lua` doesn't have any accompanying tests to verify the fix and prevent regression. 3. **Unclear impact**: Without a clear description of the bug and steps to reproduce, it's difficult to validate whether this change is correct and doesn't introduce side effects. Since this has been inactive for several months, I'm going to close this PR. If you're still experiencing the issue, please: 1. Open an issue first describing the problem with reproduction steps 2. Submit a new PR with the fix and test cases that demonstrate the before/after behavior Thanks for your interest in improving APISIX! -- 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]
