Skip to content

Commit

Permalink
Update ClientSettingsPolicy future work section (nginxinc#1785)
Browse files Browse the repository at this point in the history
Update the ClientSettingsPolicy future work section 
to include supporting attaching to GRPCRoutes 
and adding HTTP/2 and HTTP/3 directives.

Problem: The ClientSettingsPolicy didn't address 
GRPCRoute attachment or HTTP/2 and HTTP/3 directives.

Solution: Add GRPCRoute attachment and HTTP/2 and HTTP/3 
directives to the future work section of the ClientSettingsPolicy.
  • Loading branch information
kate-osborn authored and amimimor committed Apr 3, 2024
1 parent 97e11f5 commit 84c8c26
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions docs/proposals/client-settings.md
Original file line number Diff line number Diff line change
Expand Up @@ -411,6 +411,8 @@ All fields in the `ClientSettingsPolicy` will be validated with Open API Schema.

- Add support for more client-related directives, such as `client_body_buffer_size`, `client_header_buffer_size`, or `keepalive_disable`.
- Extend implementation to support multiple Gateways.
- Allow attaching to GRPCRoutes. [GRPCRoute implementation](https://github.com/nginxinc/nginx-gateway-fabric/issues/1139) is scheduled for the 1.3 release. All the directives included in this policy are applicable to gRPC servers (HTTP/2 server). For streams, the `client_max_body_size` directive applies to the entire stream, not individual messages. Therefore, for streaming methods it is recommended that this be set to a large value or 0 to disable checking.
- Extend with HTTP/2 and HTTP/3 directives. For example, `http2_preread_size`, `http2_chunk_size`, `http2_max_concurrent_streams`, etc. See the [HTTP/2](https://nginx.org/en/docs/http/ngx_http_v2_module.html) and [HTTP/3](https://nginx.org/en/docs/http/ngx_http_v3_module.html) modules for more directives.
- Add more attachment points. For example, allowing attachment to GatewayClasses or Gateway Listeners.
- Improve on status and discoverability.

Expand Down

0 comments on commit 84c8c26

Please sign in to comment.