Yep. Originally I intended to recreate the search client for each request but figured out it was unnecessary, I'll tidy this up.

Sent from my iPhone

> On Mar 12, 2014, at 4:37 PM, Jeff Handley <notifications@github.com> wrote:
> 
> In src/NuGetGallery/Infrastructure/Lucene/ExternalSearchService.cs:
> 
> > +            ServiceUri = config.SearchServiceUri;
> > +            Trace = diagnostics.SafeGetSource("ExternalSearchService");
> > +
> > +            // Extract credentials
> > +            var userInfo = ServiceUri.UserInfo;
> > +            if(!String.IsNullOrEmpty(userInfo)) {
> > +                var split = userInfo.Split(':');
> > +                if(split.Length != 2) {
> > +                    throw new FormatException("Invalid user info in SearchServiceUri!");
> > +                }
> > +
> > +                // Split the credentials out
> > +                _credentials = new NetworkCredential(split[0], split[1]);
> > +                ServiceUri = new UriBuilder(ServiceUri)
> > +                {
> > +                    UserName = null,
> Looks like _credentials is used immediately below on the SearchClient.
> 
> —
> Reply to this email directly or view it on GitHub.