Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the logic of GoAway frame handling in the client side #1833

Merged
merged 58 commits into from
Jan 29, 2024

Update stale eviction variables

29f2e6e
Select commit
Loading
Failed to load commit list.
Merged

Improve the logic of GoAway frame handling in the client side #1833

Update stale eviction variables
29f2e6e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 29, 2024 in 1s

65.97% of diff hit (target 81.68%)

View this Pull Request on Codecov

65.97% of diff hit (target 81.68%)

Annotations

Check warning on line 376 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/common/states/Http2StateUtil.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/common/states/Http2StateUtil.java#L375-L376 

Added lines #L375 - L376 were not covered by tests

Check warning on line 82 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ChannelPool.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ChannelPool.java#L82 

Added line #L82 was not covered by tests

Check warning on line 92 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ChannelPool.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ChannelPool.java#L92 

Added line #L92 was not covered by tests

Check warning on line 107 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ChannelPool.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ChannelPool.java#L107 

Added line #L107 was not covered by tests

Check warning on line 196 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ConnectionManager.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/http2/Http2ConnectionManager.java#L196 

Added line #L196 was not covered by tests

Check warning on line 125 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/EntityBodyReceived.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/EntityBodyReceived.java#L124-L125 

Added lines #L124 - L125 were not covered by tests

Check warning on line 130 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/EntityBodyReceived.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/EntityBodyReceived.java#L129-L130 

Added lines #L129 - L130 were not covered by tests

Check warning on line 148 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java#L148 

Added line #L148 was not covered by tests

Check warning on line 150 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java#L150 

Added line #L150 was not covered by tests

Check warning on line 154 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java#L154 

Added line #L154 was not covered by tests

Check warning on line 156 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/ReceivingHeaders.java#L156 

Added line #L156 was not covered by tests

Check warning on line 145 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java#L145 

Added line #L145 was not covered by tests

Check warning on line 147 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java#L147 

Added line #L147 was not covered by tests

Check warning on line 150 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java#L149-L150 

Added lines #L149 - L150 were not covered by tests

Check warning on line 154 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java#L154 

Added line #L154 was not covered by tests

Check warning on line 156 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java#L156 

Added line #L156 was not covered by tests

Check warning on line 159 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingEntityBody.java#L158-L159 

Added lines #L158 - L159 were not covered by tests

Check warning on line 148 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java#L148 

Added line #L148 was not covered by tests

Check warning on line 151 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java#L150-L151 

Added lines #L150 - L151 were not covered by tests

Check warning on line 155 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java#L155 

Added line #L155 was not covered by tests

Check warning on line 158 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingHeaders.java#L157-L158 

Added lines #L157 - L158 were not covered by tests

Check warning on line 114 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java#L114 

Added line #L114 was not covered by tests

Check warning on line 116 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java#L116 

Added line #L116 was not covered by tests

Check warning on line 120 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java#L120 

Added line #L120 was not covered by tests

Check warning on line 123 in native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/transport/contractimpl/sender/states/http2/SendingRstFrame.java#L122-L123 

Added lines #L122 - L123 were not covered by tests
View more details on Codecov