Closed k-popov closed 3 years ago
Thanks for your contribution! I appreciate it! :)
One thing to notice, In #23 there was a fix to the api_client.py:
configuration = Configuration()
api_client = ApiClient()
...
api_client = ApiClient(configuration=configuration)
CoreV1Api = client.CoreV1Api(api_client=api_client)
RbacAuthorizationV1Api = client.RbacAuthorizationV1Api(api_client=api_client)
api_temp = ApiClientTemp(configuration=configuration)
I saw that Configuration()
already set the default localhost so I there is no need to use:
api_temp = ApiClientTemp(configuration=client.configuration.Configuration.get_default_copy())
If you think I am wrong, let me know.
What does this PR do?
First issue: Appears when running
python3 KubiScan.py -rar
on local machine against Azure AKS regardless of specifying-ho
and-t
or using context from local~/.kube/config
Second issue: kubernetes API client is always trying to connect to
localhost
port80
when running on local machine against remote K8S cluster in both cases: setting-ho
+-t
and using context from local~/.kube/config
(both default context and specified explicitly)This pull request is similar to #23 but is IMO shorter and a bit more effective (
target_rules
are not uselessly iterated).Checklists
Change log
Test coverage
Documentation
README
s) were updated in this PR, and/or there is a follow-on issue to update docs, or