search

aws-samples / aws-iot-securetunneling-localproxy

AWS Iot Secure Tunneling local proxy reference C++ implementation
https://docs.aws.amazon.com/iot/latest/developerguide/what-is-secure-tunneling.html
Apache License 2.0
73 stars 70 forks source link

websocket write bug fix #88

Closed RogerZhongAWS closed 2 years ago

RogerZhongAWS commented 2 years ago

Motivation

Fixes bug where there is more than one outstanding call to web socket write.

Modifications

Change summary

Added mutex + lock_guard to protect outgoing message queue from concurrent access.

Testing

Changes have been tested with files >10 MB

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

RogerZhongAWS commented 2 years ago

@kareali I made sure the repo settings only allow for squash and merge for pull requests. Github should take care of the squashing and then I just edit the commit message before I close the PR.

kareali commented 2 years ago

@kareali I made sure the repo settings only allow for squash and merge for pull requests. Github should take care of the squashing and then I just edit the commit message before I close the PR.

Sounds good, Just approved the PR.