PaloAltoNetworks / prismacloud-api-python

Prisma Cloud utility scripts, and a Python SDK for Prisma Cloud APIs.
ISC License
60 stars 54 forks source link

can even import the package successfully on windows #146

Open sinceronny opened 1 year ago

sinceronny commented 1 year ago

Describe the bug

can't event import your python package successfully

from prismacloud.api import pc_api

Expected behavior

should import your package successfully

Current behavior

Traceback (most recent call last): File "c:\code\pl-docker-infrastructure\production\aws\PrismaAutomation\PrismaTest.py", line 2, in from prismacloud.api import pc_api File "C:\code\pl-docker-infrastructure.venv\lib\site-packages\prismacloud\api__init__.py", line 5, in from .pc_lib_api import PrismaCloudAPI File "C:\code\pl-docker-infrastructure.venv\lib\site-packages\prismacloud\api\pc_lib_api.py", line 9, in from .pc_lib_utility import PrismaCloudUtility File "C:\code\pl-docker-infrastructure.venv\lib\site-packages\prismacloud\api\pc_lib_utility.py", line 26, in
class PrismaCloudUtility(): File "C:\code\pl-docker-infrastructure.venv\lib\site-packages\prismacloud\api\pc_lib_utility.py", line 29, in PrismaCloudUtility CONFIG_DIRECTORY = os.path.join(os.environ['HOME'], '.prismacloud') File "C:\Program Files\Python39\lib\os.py", line 679, in getitem raise KeyError(key) from None KeyError: 'HOME'

Possible solution

Steps to reproduce

  1. create a python file
  2. import your package from prismacloud.api import pc_api
  3. run the script

Screenshots

Context

Your Environment

welcome-to-palo-alto-networks[bot] commented 1 year ago

:tada: Thanks for opening your first issue here! Welcome to the community!

tkishel commented 1 year ago

This mifght be solved via one of these answers:

https://stackoverflow.com/questions/14742064/python-os-environhome-works-on-idle-but-not-in-a-script

Replace os.environ['HOME'] with os.path.expanduser('~')

Or set HOME=%USERPROFILE%

Feel free to submit a PR for the former, and note that this SDK has not been developed or tested on Windows.