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

Conversation

Tratcher
Copy link
Member

@Tratcher Tratcher commented May 31, 2017

This only has internal consumers.

Fixes #1098

@Tratcher Tratcher added this to the 2.0.0-preview2 milestone May 31, 2017
@Tratcher Tratcher self-assigned this May 31, 2017
@davidfowl
Copy link
Member

Shouldn't this go into Common with the rest of the source packages?

@Tratcher
Copy link
Member Author

It can't be tested in common, it needs Logging and Configuration.

@davidfowl
Copy link
Member

Ugh this is so arbitrary.

@Tratcher
Copy link
Member Author

The other idea was to put it in Kestrel's repo so it's at least next to one of its consumers. We've done something similar for StaticFiles & MVC

@Tratcher
Copy link
Member Author

Tratcher commented Jun 1, 2017

Note types need to be made internal now.

@Tratcher Tratcher force-pushed the tratcher/certloader branch from 0750438 to 6863e26 Compare June 1, 2017 15:42
@Tratcher
Copy link
Member Author

Tratcher commented Jun 1, 2017

Will rebase onto rel/2.0.0-preview2 before committing

Copy link
Contributor

@JunTaoLuo JunTaoLuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll go with this now. Potentially split this into its own repo, or move it to kestrel.

@Tratcher
Copy link
Member Author

Tratcher commented Jun 2, 2017

Merged to rel and dev.

@Tratcher Tratcher closed this Jun 2, 2017
@Tratcher Tratcher deleted the tratcher/certloader branch June 2, 2017 21:17
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.

5 participants