BinaryTranscoder Problem

surge · November 20, 2017, 8:45pm

The BinaryTranscoder class does not have a constructor that does not accept params of IByteConverter and ITypeSerializer.

github.com

couchbase/couchbase-net-client/blob/master/Src/Couchbase/Core/Transcoders/BinaryTranscoder.cs#L15


using Couchbase.Core.Serialization;

using Couchbase.IO;

using Couchbase.IO.Converters;



namespace Couchbase.Core.Transcoders

{

/// <summary>

/// A binary level transcoder that can serialize / deserialize binary arrays using <see cref="BinaryFormatter"/>.

/// </summary>

/// <remarks>Any custom classes must have the <see cref="SerializableAttribute"/>.</remarks>

public class BinaryTranscoder : DefaultTranscoder

{

    private readonly BinaryFormatter _formatter = new BinaryFormatter();



    public BinaryTranscoder()

    {

    }



    public BinaryTranscoder(IByteConverter converter)

        : base(converter)

    {

So it fails when the TranscoderFactory attempts to create a new instance. Adding the two constructor overloads and passing them to the base class will fix the exception (missingmethod or targetinvocation, can’t remember and recalling off the top of my head).

github.com

couchbase/couchbase-net-client/blob/master/Src/Couchbase/Core/Transcoders/TranscoderFactory.cs#L55


       /// <returns>A <see cref="Func{ITypeTranscoder}"/> for creating <see cref="ITypeTranscoder"/>s.</returns>
       public static Func<ITypeTranscoder> GetTranscoder(ClientConfiguration config, string typeName)
       {
           var parameters = new object[]
           {
               config.Converter(),
               config.Serializer()
           };


           var type = Type.GetType(typeName, true);
           return () => (ITypeTranscoder)Activator.CreateInstance(type, parameters);
       }
   }
}


#region [ License information          ]


/* ************************************************************
*
*    @author Couchbase <info@couchbase.com>
*    @copyright 2017 Couchbase, Inc.

Needs same as the BinaryToJsonTranscoder:

github.com

couchbase/couchbase-net-client/blob/master/Src/Couchbase/Core/Transcoders/BinaryToJsonTranscoder.cs#L25


public class BinaryToJsonTranscoder : DefaultTranscoder
{
    public BinaryToJsonTranscoder()
    {
    }


    public BinaryToJsonTranscoder(IByteConverter converter) : base(converter)
    {
    }


    public BinaryToJsonTranscoder(IByteConverter converter, ITypeSerializer serializer) : base(converter, serializer)
    {
    }


    /// <summary>
    /// Deserializes using <see cref="BinaryFormatter"/> - does NOT deserialize to JSON!!!!
    /// </summary>
    /// <typeparam name="T"></typeparam>
    /// <param name="buffer">The buffer.</param>
    /// <param name="offset">The offset.</param>
    /// <param name="length">The length.</param>

Thanks

jmorris · November 20, 2017, 10:39pm

@surge

Thanks for reporting, here is the ticket:; the patch is on gerrit for review. It will be released in the 2.5.3 version of the SDK during the first week of December.

-Jeff

Topic		Replies	Views
RawBinaryTranscoder problem .NET SDK	5	1194	November 24, 2020
Couchbase Bucket does not use configured transcoder .NET SDK	3	2165	December 16, 2014
Unable to use LegacyTranscoder or any of the other Transcoders .NET SDK	2	790	May 29, 2020
Is the custom ITypeSerializer implementation in the docs safe to use in Prod? .NET SDK	5	353	January 31, 2024
Visual Studio Community (Mac) problems with Configuration / Startup: TypeLoadException .NET SDK	2	954	October 1, 2019

BinaryTranscoder Problem

Related topics