The Official DocuSign Python Client Library used to interact with the eSign REST API. Send, sign, and approve documents using this client. https://www.docusign.com/devcenter
MIT License
98
stars
88
forks
source link
Fix setting the Authorization header in `set_access_token` #173
In client.api_client, there is a method named set_access_token. This method takes an access token to set in the HTTP Authorization header that is sent to the Docusign API.
In integrating the Docusign API and using this SDK, specifically using the Authorization Code Grant authentication method, it was discovered that this method is incorrectly setting the Authorization header. It is simply setting the access token as the whole header value, rather than using the documented form of "Bearer <access_token>".
This PR patches the method to add the word Bearer, bringing the method up to specification in the documentation.
I could not find any tests to run that where this method is called, but if there are any, I'll gladly run them and fix them as needed.
In
client.api_client, there is a method namedset_access_token. This method takes an access token to set in the HTTPAuthorizationheader that is sent to the Docusign API.In integrating the Docusign API and using this SDK, specifically using the Authorization Code Grant authentication method, it was discovered that this method is incorrectly setting the
Authorizationheader. It is simply setting the access token as the whole header value, rather than using the documented form of"Bearer <access_token>".This PR patches the method to add the word
Bearer, bringing the method up to specification in the documentation.I could not find any tests to run that where this method is called, but if there are any, I'll gladly run them and fix them as needed.
Documentation showing the use of
Bearerhttps://developers.docusign.com/platform/auth/authcode/authcode-get-token/