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

Call DataProtectionInitializer before server starts #1096

Closed
wants to merge 1 commit into from

Conversation

natemcmaster
Copy link
Contributor

Initialize data protection during WebHost.Initialize()

Resolves aspnet/DataProtection#233

@natemcmaster
Copy link
Contributor Author

aspnet/DataProtection#239

Copy link
Member

@davidfowl davidfowl left a comment

Choose a reason for hiding this comment

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

This layering is wrong IMO, special casing data protection is hosting sets a bad precedent. Why not put it in the meta package?

@natemcmaster
Copy link
Contributor Author

We reviewed the layering and dependencies between Hosting and DP. We'll revert some changes we made in 2.0.0-preview1 which will allow us to keep DP stuff in DP.

@natemcmaster
Copy link
Contributor Author

See #1097

@natemcmaster natemcmaster deleted the namc/dataprotection branch August 29, 2017 16:54
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.

3 participants