gloo icon indicating copy to clipboard operation
gloo copied to clipboard

Add port validation when creating an upstream or a gw

Open asayah opened this issue 1 year ago • 1 comments

Gloo Edge Product

Open Source

Gloo Edge Version

latest

Is your feature request related to a problem? Please describe.

Issue:

[2023-11-13 20:09:23.209][7][warning][config] [external/envoy/source/common/config/grpc_subscription_impl.cc:126] gRPC config for type.googleapis.com/envoy.config.cluster.v3.Cluster rejected: Proto constraint validation failed (ClusterValidationError.LoadAssignment: embedded message failed validation | caused by ClusterLoadAssignmentValidationError.Endpoints[0]: embedded message failed validation | caused by LocalityLbEndpointsValidationError.LbEndpoints[0]: embedded message failed validation | caused by LbEndpointValidationError.Endpoint: embedded message failed validation | caused by EndpointValidationError.Address: embedded message failed validation | caused by AddressValidationError.SocketAddress: embedded message failed validation | caused by SocketAddressValidationError.PortValue: value must be less than or equal to 65535): name: "something"

Describe the solution you'd like

Add port validation to the gateway and upsteam object, so the port value must be less than or equal to 65535

Describe alternatives you've considered

No response

Additional Context

No response

asayah avatar Nov 13 '23 20:11 asayah