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 2021/11/18 03:51:44 UTC

[GitHub] [apisix] spacewander commented on a change in pull request #5540: 🐞 fix(plugin): fix str concat error

spacewander commented on a change in pull request #5540:
URL: https://github.com/apache/apisix/pull/5540#discussion_r751886067



##########
File path: apisix/plugin.lua
##########
@@ -264,7 +264,7 @@ function _M.load(config)
         local_conf, err = core.config.local_conf(true)
         if not local_conf then
             -- the error is unrecoverable, so we need to raise it
-            error("failed to load the configuration file: ", err)
+            error("failed to load the configuration file: " .. err)

Review comment:
       We need to raise error here as it is unrecoverable. Otherwise, APISIX is still in trouble.




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