rollbar / terraform-provider-rollbar

Terraform provider for Rollbar
https://rollbar.com
MIT License
28 stars 13 forks source link

add mutex to read/write http headers #349

Closed pawelsz-rb closed 1 year ago

pawelsz-rb commented 1 year ago

Description of the change

Adding a proper mutex to read/write http headers.

Note for the reviewer: I tried to optimize the code by locking/unlocking the mutex explicitly exactly when it was needed (overall execution little bit faster), instead of deferring unlocking, which would make the code more elegant and easier to read. (overall execution little bit slower). Please let me know which way you would prefer.

Type of change

Related issues

Shortcut stories and GitHub issues (delete irrelevant)

Checklists

Development

Code review

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 1.11.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: