envoyproxy/envoy

add listener message validation based on semantic

Open

#15,372 建立於 2021年3月8日

在 GitHub 查看
 (8 留言) (0 反應) (1 負責人)C++ (5,373 fork)batch import
area/configurationarea/listenerhelp wanted

倉庫指標

Star
 (27,997 star)
PR 合併指標
 (平均合併 8天) (30 天內合併 378 個 PR)

描述

We have tcp listener, udp listener, api listener in the same Listener message , and I am adding another internal listener.

These listeners have their own feature set. e.g. connection_balance_config should not be used by udp listener, filter_chains should only be used by tcp listener (and quic ?), not all sock options are for udp, vice versa.

We don't have adequate validation for the above implicit rules for now.

It's probably good to warn the user upon receiving listener config, starting from "some fields are contradicting or ignored" and eventually reject inconsistent config.

WDYT?

貢獻者指南