armeria
armeria copied to clipboard
Add option to respect the marshaller specified in gRPC MethodDescriptor
Motivation:
- This PR adds option to use marshaller that specified in gRPC
MethodDescriptor - Related Issue #5103
- Partial solved: Option to use marshaller that specified in gRPC
MethodDescriptor - Unsolved part: Provide way to add custom marshaller
- Partial solved: Option to use marshaller that specified in gRPC
Modifications:
- New option
useMethodMarshaller- default value is
false
- default value is
- Add validate logic for
GrpcServiceBuilderandGrpcClientBuilderto check thatunsafeWrapDeserializedBufferanduseMethodMarshallerare mutually exclusive
Result:
- Have new option
useMethodMarshaller - Throw
IllegalStateExceptionwhen bothunsafeWrapDeserializedBufferanduseMethodMarshallerare enabled
๐ Build Scanยฎ (commit: 33b67a44b0866d5bd4233a5d3874fca010214020)
| Job name | Status | Build Scanยฎ |
|---|---|---|
| build-windows-latest-jdk-21 | โ | https://ge.armeria.dev/s/c3t44iglkfbhy |
| build-self-hosted-unsafe-jdk-8 | โ | https://ge.armeria.dev/s/4r3nstqem5kew |
| build-self-hosted-unsafe-jdk-21-snapshot-blockhound | โ | https://ge.armeria.dev/s/gkmvjbkvtpeyi |
| build-self-hosted-unsafe-jdk-17-min-java-17-coverage | โ | https://ge.armeria.dev/s/ixyeas4apsboo |
| build-self-hosted-unsafe-jdk-17-min-java-11 | โ | https://ge.armeria.dev/s/dfrsbi6sxrzok |
| build-self-hosted-unsafe-jdk-17-leak | โ | https://ge.armeria.dev/s/i3ke73bcj5x72 |
| build-self-hosted-unsafe-jdk-11 | โ | https://ge.armeria.dev/s/kwk5fwzm2am4o |
| build-macos-12-jdk-21 | โ | https://ge.armeria.dev/s/zagkltchhibgs |
@jaeseung-bae ๐ ๐ ๐