Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-111037

Add the extra call options to grpc calls/streams

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: P2: Important P2: Important
    • 6.6.0 FF
    • None
    • GRPC
    • None
    • 21
    • dab59839f (dev), 3b8b24c7e (dev), aed606e6c (dev), 423568428 (dev), 9592fb0ca (dev), d5803c091 (dev), 5b5b752cb (dev), f57775c83 (dev), 6891bca29 (dev), afda2fd49 (6.6)
    • Foundation Sprint 81, Foundation Sprint 82, Foundation Sprint 83

      gRPC calls should have an extra argument that specifies the "call" options. This will allow to use a specific credentials or add some extra channel parameters(like http2 headers) for the specific call/stream only. The argument should be uninitialized by default.

      The signature of the corresponding QAbstractGrpcClient functions should be changed as following:

      QGrpcStatus call(QLatin1StringView method, const QProtobufMessage &arg, ReturnType *ret, QGrpcOptions opt = {})
      
      std::shared_ptr<QGrpcCallReply> call(QLatin1StringView method, const QProtobufMessage &arg, QGrpcOptions opt = {})
      
      std::shared_ptr<QGrpcStream> stream(QLatin1StringView method, const QProtobufMessage &arg, QGrpcOptions opt = {})
      
      std::shared_ptr<QGrpcStream> stream(QLatin1StringView method, const QProtobufMessage &arg, const QWeakPointer<ReturnType> ret, QGrpcOptions opt = {})

      The QGrpcOptions should be variation of variant map or similar structure, so channels can extract the data that is relevant for them.

      It should be possible to read the options back using classes derived from QGrpcOperation.

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            kokujawa Konrad Kujawa (Inactive)
            semlanik Alexey Edelev
            Vladimir Minenko Vladimir Minenko
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes