Skip to content

Conversation

@mhegazy
Copy link
Contributor

@mhegazy mhegazy commented Mar 13, 2017

#14594 was merged in into maser and not master.

mjbvz and others added 2 commits March 10, 2017 22:13
While working on microsoft/vscode#22437, I believe there is a bug in the per request cancellation in the  `setRequest` function on the line `currentRequestId = currentRequestId ;` This causes `currentRequestId` to always be undefined

Fix is to assign the `currentRequestId` to `requestId`
@mhegazy mhegazy merged commit 84d5d44 into master Mar 13, 2017
@mhegazy mhegazy deleted the port14594ToMaster branch March 13, 2017 17:48
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
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.

4 participants