Skip to content

Add ability to reset request headers on client attribute #318

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

Merged
merged 1 commit into from
Jun 1, 2017

Conversation

w-
Copy link
Contributor

@w- w- commented May 31, 2017

Because of the way SendGridAPIClient and python_http_client are implemented, request headers are always cached. This becomes a problem when executing a sequence of API requests that require varying request header values using the same SendGridAPIClient instance.
This method allows the caller to reset the request headers on the client instance to their default.

Because of the way SendGridAPIClient and python_http_client are implemented, request headers are always cached. This becomes a problem when executing a sequence of API requests that require varying request header values using the same SendGridAPIClient instance.
This method allows the caller to reset the request headers on the client instance to their default.
@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label May 31, 2017
@w-
Copy link
Contributor Author

w- commented Jun 1, 2017

hi @thinkingserious ,

Possible to get an ETA on when you might be able to review and hopefully merge this PR?
Thanks

@thinkingserious
Copy link
Contributor

Hello @w-,

I don't have a firm ETA, but this item is fairly high on our backlog. I don't expect it to sit for more than a week.

@w-
Copy link
Contributor Author

w- commented Jun 1, 2017

@thinkingserious
Ok thanks for the response.

@thinkingserious thinkingserious merged commit cd4d449 into sendgrid:master Jun 1, 2017
@thinkingserious
Copy link
Contributor

Hello @w-,

Thanks again for the PR!

We appreciate your contribution and look forward to continued collaboration. Thanks!

Team SendGrid DX

thinkingserious added a commit that referenced this pull request Jun 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: code review request requesting a community code review or review from Twilio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants