search

meraki / dashboard-api-python

Official Dashboard API library (SDK) for Python
MIT License
293 stars 154 forks source link

Update Python SDK to v1.6.0 (Unicode error) #144

Closed iamdexterpark closed 3 years ago

iamdexterpark commented 3 years ago

Hello,

It looks like the update to SDK v1.6.0 replaced a trademark character which is causing unicode issues:

Traceback (most recent call last): File "audit.py", line 1, in import meraki File "/$MY_PATH/venv/lib/python3.7/site-packages/meraki/init.py", line 5, in from .rest_session import * File "/$MY_PATH/venv/lib/python3.7/site-packages/meraki/rest_session.py", line 13, in from .init import version File "/$MY_PATH/venv/lib/python3.7/site-packages/meraki/init.py", line 9, in from .api.appliance import Appliance File "/$MY_PATH/venv/lib/python3.7/site-packages/meraki/api/appliance.py", line 1483 """ SyntaxError: (unicode error) 'utf-8' codec can't decode byte 0xae in position 284: invalid start byte (venv)

Please replaceSNORT® with SNORT(r) on line 1482 for:

meraki/aio/api/appliance.py meraki/api/appliance.py

Thanks in advance!

iamdexterpark commented 3 years ago

I've opened a pull request to address this issue: https://github.com/meraki/dashboard-api-python/pull/145

Pending review. Thanks in advance!

TKIPisalegacycipher commented 3 years ago

Thank you. This is fixed; I also resolved this within the generate_library.py module to help avoid this in the future. https://github.com/meraki/dashboard-api-python/commit/c9928d646b51a1ab25d680732d038af3240c058a