shreemaan-abhishek commented on PR #13780:
URL: https://github.com/apache/apisix/pull/13780#issuecomment-5203165794

   Merged `master` now that #13773 has landed, and resolved the expected 
conflict in `t/plugin/ai-aws-content-moderation.t` (my blocks renumbered to 
TEST 67-70, none of #13773's touched).
   
   Since the AWS plugin is role-selective as of #13773, the AWS coverage is now 
meaningful rather than incidental:
   
   - TEST 67/68: with `request_check_roles: ["user", "tool", "system"]`, a 
harmful `developer` message is blocked.
   - TEST 69/70: with `["user"]`, `developer` is skipped, and `grep_error_log` 
confirms only the user text reached Comprehend.
   
   The `request_check_roles` rows in the `ai-aws-content-moderation` docs (EN + 
ZH) that #13773 added now carry the same `developer` clause as the aliyun ones. 
Both moderation suites and `make lint` pass locally.


-- 
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]

Reply via email to