1# Decision Log for AWS CRT HTTP Client 2 3Note: The decision log process was implemented late in this project, so decisions earlier than 8/24/20 are not included 4below. 5 6## Log Entry Template 7 8**Source:** (Meeting/aside/pair programming discussion/daily standup) to (discuss/implement) X 9 10**Attendees:** Anna-Karin, Ben, Dongie, Irene, Matt, Nico, Vinod, John, Zoe 11 12**Closed Decisions:** 13 141. Question? Decision. Justification. 15 16**Open Decisions:** 17 181. (Old/Reopened/New) Question? 19 20## 8/24/20 21 22**Source:** Meeting to review the API surface-area of [AWS CRT HTTP Client](https://github.com/aws/aws-sdk-java-v2/tree/aws-crt-dev-preview/http-clients) 23 24**Attendees:** Anna-Karin, Ben, Dongie, John, Matt, Nico, Vinod, Zoe 25 26**Closed Decisions:** 27 281. We should add the static factory `create` methods in all HTTP Clients. It's convenient and consistent. 292. We should make the configuration classes we expose follow our SDK conventions to be consistent with what we do elsewhere 303. We should consider renaming `initialWindowSize` because it can be confused with HTTP/2 initial window size. Possible option: 31`readBufferSize` 324. We should consider renaming `httpMonitoringOptions` because it does more than monitoring. Possible option: `connectionHealthConfig` 335. We should add the service loader class. It's consistent with the way other HTTP clients work and it'll be backwards incompatible to add one later 346. We should test and support cancelling the HTTP request futures. It's supported in the Netty Http Client 357. Follow up on the features that are yet to be supported (post-preview) 36 1. Proxy TLS trust store configuration & TLS mutual auth 37 2. HTTP/2 support 38 3. Exposing connection pooling metrics 39 40**Open Decisions:** 41 42None