diff options
author | Kegan Dougal <kegan@matrix.org> | 2016-11-10 14:49:26 +0000 |
---|---|---|
committer | Kegan Dougal <kegan@matrix.org> | 2016-11-10 14:49:26 +0000 |
commit | 2771447c29402e1f26bb45bdb730bdd0fe02682f (patch) | |
tree | 76efee1716361d704b6c00d8e359cd30deaac830 /synapse/server.py | |
parent | Merge pull request #1619 from matrix-org/erikj/pwd_provider_error (diff) | |
download | synapse-2771447c29402e1f26bb45bdb730bdd0fe02682f.tar.xz |
Store Promise<Response> instead of Response for HTTP API transactions
This fixes a race whereby: - User hits an endpoint. - No cached transaction so executes main code. - User hits same endpoint. - No cache transaction so executes main code. - Main code finishes executing and caches response and returns. - Main code finishes executing and caches response and returns. This race is common in the wild when Synapse is struggling under load. This commit fixes the race by: - User hits an endpoint. - Caches the promise to execute the main code and executes main code. - User hits same endpoint. - Yields on the same promise as the first request. - Main code finishes executing and returns, unblocking both requests.
Diffstat (limited to 'synapse/server.py')
0 files changed, 0 insertions, 0 deletions