Skip to content

Add ability to reset request headers on client attribute#318

Merged
thinkingserious merged 1 commit intosendgrid:masterfrom
w-:reset_request_headers
Jun 1, 2017
Merged

Add ability to reset request headers on client attribute#318
thinkingserious merged 1 commit intosendgrid:masterfrom
w-:reset_request_headers

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