Skip to content

Reverts calling next() after handling response #2634

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Sep 9, 2016

Conversation

flovilmart
Copy link
Contributor

Reverts #2559

@ghost
Copy link

ghost commented Sep 1, 2016

@flovilmart updated the pull request - view changes

@flovilmart flovilmart force-pushed the revert-2559-routing-improvements branch from 82f947a to bd3b512 Compare September 1, 2016 04:09
@ghost
Copy link

ghost commented Sep 1, 2016

@flovilmart updated the pull request - view changes

@codecov-io
Copy link

codecov-io commented Sep 1, 2016

Current coverage is 92.22% (diff: 100%)

Merging #2634 into master will decrease coverage by 0.03%

@@             master      #2634   diff @@
==========================================
  Files           102        102          
  Lines         12450      12446     -4   
  Methods        1557       1559     +2   
  Messages          0          0          
  Branches       2043       2040     -3   
==========================================
- Hits          11486      11478     -8   
- Misses          964        968     +4   
  Partials          0          0          

Powered by Codecov. Last update 7f6fb11...a482019

@flovilmart
Copy link
Contributor Author

flovilmart commented Sep 1, 2016

Fixes #2362
Fixes #2580

and probably other issues...

Happy to discuss here. @blacha if you're ok with it, we'll move forward removing that next() call, even I don't like not having it, but if it's painful for everyone, let's stop the pain.

@flovilmart flovilmart force-pushed the revert-2559-routing-improvements branch from bd3b512 to 6c4e7da Compare September 3, 2016 13:52
@facebook-github-bot
Copy link

@flovilmart updated the pull request - view changes

@flovilmart flovilmart changed the title Revert "Makes sure routes don't overlap and yield a header set error" Reverts calling next() after handling response Sep 3, 2016
@flovilmart flovilmart added this to the 2.2.19 milestone Sep 4, 2016
@drew-gross
Copy link
Contributor

Could we get a regression test on this? Seem like is issue is a tricky one.

@ghost
Copy link

ghost commented Sep 9, 2016

@flovilmart updated the pull request - view changes

@flovilmart flovilmart force-pushed the revert-2559-routing-improvements branch from 924686b to a482019 Compare September 9, 2016 21:02
@flovilmart flovilmart merged commit ac348f5 into master Sep 9, 2016
@ghost
Copy link

ghost commented Sep 9, 2016

@flovilmart updated the pull request - view changes

@flovilmart flovilmart deleted the revert-2559-routing-improvements branch September 17, 2016 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants