xiazcy commented on PR #3334: URL: https://github.com/apache/tinkerpop/pull/3334#issuecomment-4111943421
> There's a bit of a design issue with the way this is currently implemented. The `HttpRequestContext` allows for setting/getting the body, but the body is immediately serialized before the interceptors run and the only supported serializer is GraphBinary. This means that users really have no chance to modify the body at all and they can only read it, otherwise, their interceptor would need to deserialize the body, make changes and then re-serialize. Updated to be similar to python, where request and response serializers are separate options and a nullable request serializer allows the request to not be serialized when null, and leave to the interceptors to serialize. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
