Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Setting Response.OnStarting should throw when response already started #806

Closed
Tratcher opened this issue May 5, 2016 · 0 comments
Closed

Setting Response.OnStarting should throw when response already started #806

Tratcher opened this issue May 5, 2016 · 0 comments
Assignees
@cesarblum
Labels
3 - Done bug
Milestone
1.1.0-preview1

Comments

@Tratcher
Copy link
Member

Tratcher commented May 5, 2016

From @pakrym on May 5, 2016 21:52

This would prevent hard to debug issues with callbacks that never ran.

Copied from original issue: aspnet/HttpAbstractions#627
@Tratcher Tratcher mentioned this issue May 5, 2016
Closed
@muratg muratg changed the title Response.OnStarting should throw when response allready started Response.OnStarting should throw when response already started May 6, 2016
@danroth27 danroth27 added this to the Backlog milestone May 9, 2016
@danroth27 danroth27 added the bug label May 9, 2016
@muratg muratg modified the milestones: 1.1.0, Backlog Jul 25, 2016
cesarblum pushed a commit that referenced this issue Aug 8, 2016
Throw when setting OnStarting after response has already started (#806).
a5aacd6
@cesarblum cesarblum mentioned this issue Aug 8, 2016
Merged
@cesarblum cesarblum changed the title Response.OnStarting should throw when response already started Setting Response.OnStarting should throw when response already started Aug 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cesarblum cesarblum

Labels
3 - Done bug
Projects
None yet
Milestone
1.1.0-preview1
Development

No branches or pull requests
4 participants
@cesarblum @muratg @Tratcher @danroth27