fix all targets of route grpc connect will be closed when update route

fabiolb / fabio

Consul Load-Balancing made simple

https://fabiolb.net

MIT License

7.25k stars 621 forks source link

fix all targets of route grpc connect will be closed when update route #808

Closed qqwx1986 closed 2 years ago

qqwx1986 commented 3 years ago

fix issue #807

cmvoicu commented 3 years ago

Can anyone take a look at this PR? The issue for droping gRPC streams on config update it's quite big, considering that the stream should be alive as long as the service is alive and is present in the registry!

CLAassistant commented 2 years ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

wangxu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}