Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

authentication: add AES support #49

Closed
wants to merge 1 commit into from
Closed

Conversation

realitix
Copy link

Currently, the Python SDK only support basic API key authentication. This patch adds the support for AES authentication which is important if you want to properly secure your requests.

Documentation is updated too.

Currently, the Python SDK only support basic API key authentication.
This patch adds the support for AES authentication which is important if
you want to properly secure your requests.

Documentation is updated too.
@YoungWing
Copy link

YoungWing commented Oct 31, 2022

@realitix Thank you for your support for API key AES signature authentication. We think this PR is a broken change of only support as-api-key. For existing user, if they upgrade SDK to this version, it means that they have to use new API-Key to access the APIs, otherwise they will get authentication fail errors. A few weeks ago, we realized the importance of AES signature, so we will support this feature before next friday. Thanks again.

@realitix
Copy link
Author

Ok @YoungWing, take my code and just update the headers in sign_request by replacing aftership-api-key to as-api-key and it will work without breaking change.

@YoungWing
Copy link

YoungWing commented Oct 31, 2022

yes. If your API-Key is created this year, it is working to use as-api-key.

@YoungWing YoungWing closed this Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants