Added option to run ADT calls over RFC

[akreuzer]

Main effort is to split Connection in sap/adt/core.py into ConnectionViaRFC and ConnectionViaHTTP.

I don't have many test for this as this would require a Netweaver Server.

[jfilak]

Good job! I was always wondering how that RFC HTTP tunneling works :) I am happy to merge once we get rid of those unnecessary style fixes and all tests and linters are satisfied.

[akreuzer]

Thank you. I must have accidentally run yapf on the changes. I should have reverted all of the cosmetic changes and also the stuff with urllib3.

[jfilak]

Please update doc/configuration.md with all the recent congiration options you have added.

[akreuzer]

• Added new options to doc/configuration.md
• Fixed on typo in bin/sapcli.
• Fixed some issue with liniting.
• Added NotImplementedException to the case where --rest-over-rfc is used with odata/gtcs. (This is just to illustrated - if you don't like it I can revert this commit.)

[lucasborin]

If you do not mind, I would like to test it. However, pulling your forked version raised a syntax error:

Besides, I added these two entries to my cfg file:

export SAPCLI_HTTP_TIMEOUT=2000
export SAP_REST_OVER_RFC=yes

Any clue?

Git stauts:

[akreuzer]

I think I know what this is: list[str] as a type definition is only allowed in Python 3.9+

For older version on has to use something like this

from typing import List
a: List[str]

@lucasborin I updated the code. Let me know if it works now.

[lucasborin]

Unsure if I did something wrong, but my python is not able to find PyRFC anymore before pulling it. I'll try to redo my setup.