envoyproxy / envoyproxy/java-control-plane

Watch Created with Empty Resource Names

オープン
#411 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
312
フォーク
149
PR マージ指標
30日以内にマージされた PR はありません

説明

According to the [Envoy xDS protocol documentation](https://www.envoyproxy.io/docs/envoy/latest/api-docs/xds_protocol#how-the-client-specifies-what-resources-to-return):
```
If the list of resource names becomes empty, that means the client is no longer interested in any resources of the specified type.
```

However, the `DiscoveryRequestStreamObserver` currently creates a new watch even when the resource names list becomes empty, which violates the spec.
https://github.com/envoyproxy/java-control-plane/blob/main/server/src/main/java/io/envoyproxy/controlplane/server/DiscoveryRequestStreamObserver.java#L80-L94

I think we can just cancel the previous watch and not create another one in this case. Please let me know if this makes sense, and I’ll be happy to submit a PR.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。