egg-grpc
egg-grpc copied to clipboard
新增 grpc ssl 安全配置
Checklist
- [ ]
npm test
passes - [ ] tests and/or benchmarks are included
- [ ] documentation is changed or added
- [ ] commit message follows commit guidelines
Affected core subsystem(s)
Description of change
新增 grpc client ssl 安全配置
Thanks for your contributing. 🎉
Please make sure that the code is checked through the checklist and pass the CI testing.
We might need some tests to test this new feature.
Otherwise, I notice the package name (which in package.json
) has been changed, we should keep the name as before as send it to npm.
It might also includes a breaking change which is unnecessary - The configuration of this plugin didn't support multiple entries so the configurations could set the first client configurations directly without clients
layer. If we want to make it supports the multiple entries we might need to keep supporting directly with one client configuration. We can check if the clients
field is exists at the first time.
reference to https://github.com/eggjs/egg/issues/4077
Thanks for your contributing. 🎉
Please make sure that the code is checked through the checklist and pass the CI testing.
We might need some tests to test this new feature.
Otherwise, I notice the package name (which in
package.json
) has been changed, we should keep the name as before as send it to npm.It might also includes a breaking change which is unnecessary - The configuration of this plugin didn't support multiple entries so the configurations could set the first client configurations directly without
clients
layer. If we want to make it supports the multiple entries we might need to keep supporting directly with one client configuration. We can check if theclients
field is exists at the first time.
https://github.com/xdxiaodong/egg-grpc-x