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 gRPC error handling for gateway/filter handler #2669

Merged
merged 4 commits into from
Oct 9, 2024

Merge branch 'main' into refactor/filter-grpc-status-handling

73aa6b8
Select commit
Loading
Failed to load commit list.
Merged

Fix gRPC error handling for gateway/filter handler #2669

Merge branch 'main' into refactor/filter-grpc-status-handling
73aa6b8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Oct 9, 2024 in 1s

0.00% of diff hit (target 23.94%)

View this Pull Request on Codecov

0.00% of diff hit (target 23.94%)

Annotations

Check warning on line 240 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L239-L240 

Added lines #L239 - L240 were not covered by tests

Check warning on line 242 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L242 

Added line #L242 was not covered by tests

Check warning on line 286 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L285-L286 

Added lines #L285 - L286 were not covered by tests

Check warning on line 288 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L288 

Added line #L288 was not covered by tests

Check warning on line 468 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L467-L468 

Added lines #L467 - L468 were not covered by tests

Check warning on line 470 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L470 

Added line #L470 was not covered by tests

Check warning on line 518 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L517-L518 

Added lines #L517 - L518 were not covered by tests

Check warning on line 520 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L520 

Added line #L520 was not covered by tests

Check warning on line 700 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L699-L700 

Added lines #L699 - L700 were not covered by tests

Check warning on line 702 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L702 

Added line #L702 was not covered by tests

Check warning on line 759 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L758-L759 

Added lines #L758 - L759 were not covered by tests

Check warning on line 761 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L761 

Added line #L761 was not covered by tests

Check warning on line 940 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L939-L940 

Added lines #L939 - L940 were not covered by tests

Check warning on line 942 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L942 

Added line #L942 was not covered by tests

Check warning on line 998 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L997-L998 

Added lines #L997 - L998 were not covered by tests

Check warning on line 1000 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1000 

Added line #L1000 was not covered by tests

Check warning on line 1187 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1186-L1187 

Added lines #L1186 - L1187 were not covered by tests

Check warning on line 1189 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1189 

Added line #L1189 was not covered by tests

Check warning on line 1245 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1244-L1245 

Added lines #L1244 - L1245 were not covered by tests

Check warning on line 1247 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1247 

Added line #L1247 was not covered by tests

Check warning on line 1355 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1354-L1355 

Added lines #L1354 - L1355 were not covered by tests

Check warning on line 1357 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1357 

Added line #L1357 was not covered by tests

Check warning on line 1438 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1437-L1438 

Added lines #L1437 - L1438 were not covered by tests

Check warning on line 1440 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1440 

Added line #L1440 was not covered by tests

Check warning on line 1527 in pkg/gateway/filter/handler/grpc/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/gateway/filter/handler/grpc/handler.go#L1526-L1527 

Added lines #L1526 - L1527 were not covered by tests
View more details on Codecov