qiurui
/
qtgrpc
mirror of https://github.com/qt/qtgrpc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove the explicit setting of the accept-encoding header 

QHttp2 implementation will make this favor for us, in general the
header is not mandatory for communication with gRPC server, but can
be used by other members of the network communication. Anyway it's
not the level of the gRPC channel where the header should be defined.

Pick-to: 6.9.0 6.9 6.8 6.8.3
Change-Id: I73fe3c780bfa373a724255dd26accf37f8a8ff50
Reviewed-by: Dennis Oberst <dennis.oberst@qt.io>
This commit is contained in:
Alexey Edelev 2025-03-05 16:34:31 +01:00 committed by Dennis Oberst
parent 81bc13aec3
commit 39f2692655
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/grpc/qgrpchttp2channel.cpp
View File

@ -516,8 +516,6 @@ void Http2Handler::prepareInitialRequest(QGrpcOperationContext *operationContext
const static QByteArray PathHeader(":path");
const static QByteArray SchemeHeader(":scheme");
const static QByteArray AcceptEncodingHeader("accept-encoding");
const static QByteArray AcceptEncodingValue("identity,gzip");
const static QByteArray TEHeader("te");
const static QByteArray TEValue("trailers");
const static QByteArray GrpcServiceNameHeader("service-name");
@ -539,7 +537,6 @@ void Http2Handler::prepareInitialRequest(QGrpcOperationContext *operationContext
{ ContentTypeHeader, channel->contentType() },
{ GrpcServiceNameHeader, service },
{ GrpcAcceptEncodingHeader, GrpcAcceptEncodingValue },
{ AcceptEncodingHeader, AcceptEncodingValue },
{ UserAgentHeader, UserAgentValue },
{ TEHeader, TEValue },
};