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

Remove dependency on DependencyInjection and instead use DependencyInjection.Interfaces on Http.Extensions #228

Closed
davidfowl opened this issue Mar 10, 2015 · 3 comments
Closed

Remove dependency on DependencyInjection and instead use DependencyInjection.Interfaces on Http.Extensions #228

davidfowl opened this issue Mar 10, 2015 · 3 comments
Assignees
@Praburaj
Labels
3 - Done bug
Milestone
1.0.0-beta4

Comments

@davidfowl
Copy link
Member

https://github.com/aspnet/HttpAbstractions/blob/dev/src/Microsoft.AspNet.Http.Extensions/project.json#L7
@davidfowl davidfowl added the bug label Mar 10, 2015
@davidfowl davidfowl added this to the 1.0.0-rc1 milestone Mar 10, 2015
@davidfowl davidfowl assigned davidfowl and Praburaj and unassigned davidfowl Mar 10, 2015
@davidfowl
Copy link
Member Author

/cc @Praburaj

@Praburaj
Copy link
Contributor

@davidfowl Http.Extensions seems to have dependency over DependencyInjection as it requires ActivatorUtilities to instantiate the middleware object in the UseMiddleware extension. Not sure if there is an alternate for this.

This was referenced Mar 13, 2015
Closed
Merged
Praburaj pushed a commit to aspnet/DependencyInjection that referenced this issue Mar 16, 2015
Remove dependency on DependencyInjection and instead use DependencyIn…
702e9a7 
…jection.Interfaces on Http.Extensions

Fixes: aspnet/HttpAbstractions#228
@Praburaj
Copy link
Contributor

Merged 93deb0b

@Praburaj Praburaj reopened this Mar 16, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Praburaj Praburaj

Labels
3 - Done bug
Projects
None yet
Milestone
1.0.0-beta4
Development

No branches or pull requests
2 participants
@davidfowl @Praburaj