Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Remove threading from http plugin, potential fix for #202 #233

Merged
merged 1 commit into from
Aug 26, 2017

Conversation

MatthewPappas
Copy link
Contributor

Please merge this to master

@MatthewPappas
Copy link
Contributor Author

I am commenting that I just clicked and commented after pull request - hmmm, so this can merge automatically, brave.

@pmesnier pmesnier self-requested a review August 26, 2017 17:21
Copy link
Contributor

@pmesnier pmesnier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm accepting this for expediency, but I think that either we strip out the thread code completely, or refactor so that we can make it optional. As it is, only 1 thread could be active anyway.

@pmesnier pmesnier merged commit 72656e5 into master Aug 26, 2017
@MatthewPappas MatthewPappas deleted the crash-202 branch August 28, 2017 05:40
taokayan pushed a commit to taokayan/eos that referenced this pull request May 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants