americanexpress / nodes

A GraphQL JVM Client - Java, Kotlin, Scala, etc.
Apache License 2.0
307 stars 70 forks source link

feat(Fetch) support keep-alive connections #87

Closed fupduck closed 2 years ago

fupduck commented 5 years ago

Currently keep-alive connections throw an IllegalStateException - already connected

This adds keep-alive support, by catching errors if already connected and not setting Content-Length explicitly.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 106


Changes Missing Coverage Covered Lines Changed/Added Lines %
nodes/src/main/java/io/aexp/nodes/graphql/Fetch.java 13 14 92.86%
<!-- Total: 13 14 92.86% -->
Totals Coverage Status
Change from base Build 104: -0.2%
Covered Lines: 540
Relevant Lines: 546

💛 - Coveralls