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
Add appsecret_proof support #243
Conversation
@martey is there any hope in getting this merged soon? It's quite important for better security of setups with mobile apps, which may pass access tokens to the server. If there's something missing in the PR I'm happy help patching it up. |
I'd help too! Docs are the most important. |
I'd like to see automated tests and updated documentation before merging this. |
@dimitrovskif Are you planning on updating this PR, or do you need help? |
Closing in favor of #321, which includes tests. |
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line Duplicated this in rebase
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line Duplicated this in rebase
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line Duplicated this in rebase
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line Duplicated this in rebase
add tests and documentation, pep8 dimitrovskif changes to add appsecret_proof support mobolic#243 use hexdigest add tests pep8 add documentation remove spaces around keyword / parameter equals python 2 conditional install in one line Duplicated this in rebase
Adds support for secure Graph API calls with the
appsecret_proof
GET param (or POST ifpost_args
). Described hereThis PR successfully fixes #213, which was good but rejected because it wasn't PEP8 compliant and the argument wasn't positioned correctly (new argument last for compatibility).