n8n sending invalid GET requests

[pemontto]

Describe the bug
n8n's axios library appears to send invalid GET requests with an HTTP body. While most services accept this, intermediate proxies often reject them. I have confirmed this occurs with both Github and Clockify nodes.

To Reproduce
Steps to reproduce the behavior:

1. Configure http_proxy and https_proxy environment variables to use an http proxy
2. Add a Github or Clockify node, potentially others
3. Configure a resource and operation that makes a GET request
   • Github: Resource: User, Operation: Get Repositories
   • Clockify: Resource: Workspace, Operation: Get all
4. Send the request
5. See error

Packet capture showing empty body with GET request:

Squid proxy error:

Expected behavior
GET requests should not have request bodies. Although not strictly prohibited by the spec, a GET body has no semantic meaning. Similar to Python requests library a GET request with a body should be considered invalid.

>>> requests.get(url='http://ipinfo.io', body='')
TypeError: request() got an unexpected keyword argument 'body'

Environment (please complete the following information):

• OS: MacOS 12 & Docker Alpine
• n8n Version: 0.184.0
• Node.js Version: v16.15.1
• Database system: SQLite & PostgreSQL
• Operation mode: own & queue

Additional context
None

[Joffcom]

@pemontto as you have put in a PR for this I am going to close this and we can handle this on the PR.