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

Don't expose ServiceProvider on ControllerBase #4097

Closed
davidfowl opened this issue Feb 15, 2016 · 3 comments
Closed

Don't expose ServiceProvider on ControllerBase #4097

davidfowl opened this issue Feb 15, 2016 · 3 comments
Assignees
@ryanbrandenburg
Labels
3 - Done enhancement
Milestone
1.0.0-rc2

Comments

@davidfowl
Copy link
Member

It's already available on HttpContext and we really don't want people to use it directly. Let's not expose it from multiple vectors.
@khellang
Copy link
Contributor

THIS 👍

@rynowak
Copy link
Member

rynowak commented Feb 16, 2016

FYI this was added for back-compat

@davidfowl
Copy link
Member Author

I bet, but we should drop it.

@danroth27 danroth27 added this to the 1.0.0-rc2 milestone Feb 17, 2016
@ryanbrandenburg ryanbrandenburg mentioned this issue Mar 15, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ryanbrandenburg ryanbrandenburg

Labels
3 - Done enhancement
Projects
None yet
Milestone
1.0.0-rc2
Development

No branches or pull requests
5 participants
@davidfowl @khellang @rynowak @danroth27 @ryanbrandenburg