I saw it got lost too but I was happy that question got lost since after I asked, I realized we’re not in an http request context when we’re doing this.

From: Jeff Handley [mailto:notifications@github.com]
Sent: Thursday, January 17, 2013 1:09 AM
To: NuGet/NuGetGallery
Cc: Tim Lovell-Smith
Subject: Re: [NuGetGallery] Move Secrets to .cscfg file (#726)


In Website/Entities/EntitiesContext.cs:

>      public class EntitiesContext : DbContext, IWorkItemsContext, IEntitiesContext

>      {

> -        public EntitiesContext()

> -            : base("NuGetGallery")

> +        public EntitiesContext(string connectionString)

It looks like @TimLovellSmith<https://github.com/TimLovellSmith> had a comment/question that got lost.

"Should we be doing explicit new EntitiesContext(configuration) everywhere? This feels like we are failing to use Ninject to inject the dependencies"

—
Reply to this email directly or view it on GitHub<https://github.com/NuGet/NuGetGallery/pull/726/files#r2678157>.