mapillary / mapillary-python-sdk

A Python 3 library built on the Mapillary API v4 to facilitate retrieving and working with Mapillary data.
MIT License
39 stars 15 forks source link

[SECURITY - ACCESS_TOKEN] Access Token Introduced, PR#44 #51

Closed Rubix982 closed 3 years ago

Rubix982 commented 3 years ago

Describe the bug Access token introduced under tests/utils by PR #43

To Reproduce NA

Expected behavior NA

Screenshots NA

Additional context Security concern!

To fix this, thought should be given on how when we write some tests, we can automate them. The primary concern is the access_token that I should not ever be exposed to. To fix this, I need to make my tests first set the access token, then run the tests to prevent any exposure at all. This should be a separate PR, and thus not be solely handled by #44.