Set Content-Length header in response

[acveigas]

Came across few SOAP clients which require Content-Length header to be set in order for the SOAP request to go through.
Without this change, Content-Length header is not set and Transfer-Encoding is chunked

[acveigas]

@andersjonsson Can you approve the Github actions (Pipelines)?

[andersjonsson]

Thanks for the PR!
I’ll take a closer look later today

[acveigas]

Thanks. I Will also look at the CodeQL failure later on this week.

Thanks for the PR!

I’ll take a closer look later today

[andersjonsson]

Left a few notes.

[andersjonsson]

@acveigas Damn, I need to learn to read better 😆

I thought you changed Content-Length to the number of bytes.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Length

Try sending a character that is represented by two bytes, like ü or é. Maybe your legacy client will accept whatever in the header, but I think the correct behavior would be to set the header to the number of bytes.

What do you think?

[acveigas]

I can send a PR with number of bytes. Shouldn't take time 😊

[andersjonsson]

I can send a PR with number of bytes. Shouldn't take time 😊

Thanks!