Fix #388 Triggering Authenticator double encodes body in POST, PUT, PATCH

[techouse]

This fixes #388

When ChopperClient is configured with an Authenticator and when Authenticator.authenticate returns a non-null value, ChopperClient.send recursively passes an already converted Request to ChopperClient.send again. This results in double-encoded JSON.

Expected: '{"name":"john","surname":"doe"}'
  Actual: '"{\\"name\\":\\"john\\",\\"surname\\":\\"doe\\"}"'
   Which: is different.
          Expected: {"name":"j ...
            Actual: "{\\"name\ ...
                    ^
           Differ at offset 0

As previously mentioned, the issue only manifests itself whenAuthenticator.authenticate returns a non-null value, in essence when the client receives an HTTP ERROR: 401. It works perfectly fine when the client is authenticated and/or the Auth token hasn't expired. That's why it probably flew under the radar for so long.

This PR fixes it by passing the original Request to ChopperClient.send in that case.

[JEuler]

That's so cool you've added tests! <3 Thank you!

[techouse]

That's so cool you've added tests! <3 Thank you!

Yeah, I was only able to replicate the issues with tests. After confirming the issue with tests I proceeded to fix it. TDD I guess 🙈

[JEuler]

Should I merge it or you will? :)

[techouse]

Should I merge it or you will? :)

You're the merge master 😎

[techouse]

@JEuler when are we getting this fix to origin/master?

[JEuler]

Hm, let me prepare PR tomorrow. :)

[JEuler]

What do you think about the version?
b0fd18f - I think we need to bump major version as I understand and remember.

[techouse]

Probably v5.2.0

[JEuler]

Okay, I will do it today. :)