You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/01/25 01:04:39 UTC

[GitHub] [apisix] tokers commented on a change in pull request #6151: fix(limit-count): keep the counter if the plugin conf is the same

tokers commented on a change in pull request #6151:
URL: https://github.com/apache/apisix/pull/6151#discussion_r791274013



##########
File path: t/plugin/limit-count-redis-cluster.t
##########
@@ -63,7 +63,7 @@ __DATA__
 GET /t
 --- error_code: 400
 --- response_body
-{"error_msg":"failed to check the configuration of plugin limit-count err: failed to validate dependent schema for \"policy\": value should match only one schema, but matches none"}
+{"error_msg":"failed to check the configuration of plugin limit-count err: else clause did not match"}

Review comment:
       The error messages is confusing after changes, it cannot help API callers troubleshoot effectively, will we have some enhancements? 




-- 
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: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org