zll2 commented on a change in pull request #4702:
URL: https://github.com/apache/apisix/pull/4702#discussion_r679082183
##########
File path: docs/en/latest/discovery/nacos.md
##########
@@ -38,6 +38,10 @@ discovery:
nacos:
host:
- "http://${username}:${password}@${host1}:${port1}";
+ authorization:
+ type: "basic_auth" # default basic_auth
+ access_key: "" # default empty
Review comment:
basic_auth 是原先 nacos.lua 里的分支判断，enreka 是使用了该判断，我看了下实际是不需要，就给去掉了；为了区分和
阿里云 MSE Nacos 的认证鉴权类型区分，然后就拿 basic_auth 作为默认的 type 了；开源版的 Nacos 自建的话，可以使用 用户名 及
密码进行加强安全验证，也可以不使用，但 阿里云 MSE Nacos 的商业版本在安全认证的方面仅支持 AccessKey & Access Secret 进行
OpenAPI 交互；
我看 APISIX 插件中的也是叫 basic—auth，你有更好的建议吗？
--
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]
