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

Fix error logging logic with interceptors #1846

Merged
merged 23 commits into from
Feb 1, 2024

Merge branch 'master' into fix-interceptor-error-logs

0aa0a21
Select commit
Loading
Failed to load commit list.
Merged

Fix error logging logic with interceptors #1846

Merge branch 'master' into fix-interceptor-error-logs
0aa0a21
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Feb 1, 2024 in 1s

85.48% of diff hit (target 81.33%)

View this Pull Request on Codecov

85.48% of diff hit (target 81.33%)

Annotations

Check warning on line 288 in native/src/main/java/io/ballerina/stdlib/http/api/BallerinaHTTPConnectorListener.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/BallerinaHTTPConnectorListener.java#L288 

Added line #L288 was not covered by tests

Check warning on line 120 in native/src/main/java/io/ballerina/stdlib/http/api/HttpDispatcher.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/HttpDispatcher.java#L120 

Added line #L120 was not covered by tests

Check warning on line 157 in native/src/main/java/io/ballerina/stdlib/http/api/HttpDispatcher.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/HttpDispatcher.java#L157 

Added line #L157 was not covered by tests

Check warning on line 190 in native/src/main/java/io/ballerina/stdlib/http/api/HttpDispatcher.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/HttpDispatcher.java#L189-L190 

Added lines #L189 - L190 were not covered by tests

Check warning on line 65 in native/src/main/java/io/ballerina/stdlib/http/api/ResourceDispatcher.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/ResourceDispatcher.java#L65 

Added line #L65 was not covered by tests

Check warning on line 71 in native/src/main/java/io/ballerina/stdlib/http/api/nativeimpl/ExternRequestContext.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/nativeimpl/ExternRequestContext.java#L71 

Added line #L71 was not covered by tests

Check warning on line 162 in native/src/main/java/io/ballerina/stdlib/http/api/service/signature/AllHeaderParams.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/service/signature/AllHeaderParams.java#L161-L162 

Added lines #L161 - L162 were not covered by tests

Check warning on line 106 in native/src/main/java/io/ballerina/stdlib/http/api/service/signature/ParamUtils.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/service/signature/ParamUtils.java#L104-L106 

Added lines #L104 - L106 were not covered by tests

Check warning on line 109 in native/src/main/java/io/ballerina/stdlib/http/api/service/signature/ParamUtils.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/service/signature/ParamUtils.java#L109 

Added line #L109 was not covered by tests

Check warning on line 111 in native/src/main/java/io/ballerina/stdlib/http/api/service/signature/ParamUtils.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/service/signature/ParamUtils.java#L111 

Added line #L111 was not covered by tests

Check warning on line 171 in native/src/main/java/io/ballerina/stdlib/http/api/service/signature/PayloadParam.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

native/src/main/java/io/ballerina/stdlib/http/api/service/signature/PayloadParam.java#L171 

Added line #L171 was not covered by tests
View more details on Codecov