Similarityoung commented on PR #97: URL: https://github.com/apache/dubbo-go-pixiu-samples/pull/97#issuecomment-3324563771
> 1. pls split this pr into 2 prs, i.e. `nacos` and `oauth`, this pr is too heavy to review > 2. `mcp/oauth/authserver` appears to be a standalone verification server. Move it to the tools directory and write a relevant description in the root directory's README file. I thought about it and decided it wouldn't be convenient to split this into two PRs. I'll wait until the previous OAuth PR is merged before modifying this PR. Also, for this authentication server, I'm currently only implementing an authorization server under the MCP protocol, which may be different from a general authorization server. Consider placing it in the tools location for future expansion. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
