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

Remove queued HTTP handler registration #6686

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

spoonincode
Copy link
Contributor

Change Description

websocketpp's socket handlers run at ultra-high priority which means they can beat registration of HTTP endpoints because they are queued via low priority. This is what happened in #6683 where the websocketpp code handled an incoming request but the endpoint registration was still queued up low priority

No real reason needed to queue up registration of handlers at this point so just remove this queue

Consensus Changes

API Changes

Documentation Additions

websocketpp's socket handlers run at ultra-high priority which means they can beat registration of HTTP endpoints because they are queued via low priority.

No real reason needed to queue up registration of handlers at this point so just remove this queue
@spoonincode spoonincode merged commit faece37 into develop Jan 30, 2019
@spoonincode spoonincode deleted the remove_queued_rpc_registration branch January 30, 2019 20:57
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.

2 participants