fix(sdk): Add host to headers if specified

[tomdsmartdata]

bug: #372 Support for a proxy using SSL

Description 📝

• Purpose: Implements a solution for raygun.transport: Support for a proxy using SSL
• Approach: It assigns the host header based on the client options, falling back to the constant in the class.

Type of change

Delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

Updates

👉 raygun.transport pulls host header from options.http.host

Test plan 🧪

Follow reproduction steps from raygun.transport: Support for a proxy using SSL

Author to check 👓

• Project and all contained modules builds successfully
• Self-/dev-tested
• Unit/UI/Automation/Integration tests provided where applicable
• Code is written to standards
• Appropriate documentation written (code comments, internal docs)

Reviewer to check ✔️

• Project and all contained modules builds successfully
• Change has been dev-/reviewer-tested, where possible
• Unit/UI/Automation/Integration tests provided where applicable
• Code is written to standards
• Appropriate documentation written (code comments, internal docs)

[TheRealAgentK]

Thx @tomdsmartdata !

@miquelbeltran I think this is generally a feasible approach. Might be worthwhile checking if @tomdsmartdata's change is the only place where we set headers though.

[tomdsmartdata]

Thx @tomdsmartdata !

@miquelbeltran I think this is generally a feasible approach. Might be worthwhile checking if @tomdsmartdata's change is the only place where we set headers though.

I only validated this with my application and opted for as minimal a change in the PR as I could, hence no automated tests.
If you do have other areas where headers are being set that you'd like addressed in this PR, I'm definitely open to helping contribute those changes and properly test them all.

[TheRealAgentK]

@tomdsmartdata

@miquelbeltran comes online in a few hours and he's best placed to have a proper look into it.

Thanks for your comms, it's really nice to have someone reporting an issue or asking for an improvement who's really engaged with the process! 👍

[miquelbeltran]

Thanks for the contribution @tomdsmartdata
This was a bug indeed, I will proceed to merge and prepare a new release.

Re: the failing CI job. The prettier formatting is fine, just the job seems to fail to checkout the code, I will have to check why in a different pr.

[TheRealAgentK]

Thx again @tomdsmartdata !!!