Lol. GetConfig was in there cause I was going to log to storage. Decided
writing to a blob page was to much of a pain.
On Jun 30, 2013 5:10 PM, "Håkan Edling" <notifications@github.com> wrote:

> In Piranha.Azure/Log/TraceLogProvider.cs:
>
> > +        /// <param name="details">Optional error details</param>
> > +        public void Error(string origin, string message, Exception details = null)
> > +        {
> > +            System.Diagnostics.Trace.WriteLine(string.Format("ERROR [{0}] Origin [{1}] Message [{2}]", DateTime.Now.ToString("yyyy-MM-dd HH:mm:ss"), origin, message));
> > +            if (details != null)
> > +            {
> > +                System.Diagnostics.Trace.WriteLine(details);
> > +            }
> > +        }
> > +
> > +        #region Private methods
> > +        /// <summary>
> > +        /// Gets the configuration section from the Web.config.
> > +        /// </summary>
> > +        /// <returns>The configuration</returns>
> > +        private ConfigFile GetConfig()
>
> Great work! We can move the file-cache around a bit, one solution would be
> to extend the IMediaProvider to include methods for storing cached images.
> It could also be moved to a separate provider.
>
> However I don't really get where the GetConfig() method in the
> TraceLogProvider is called from, but maybe the pieces will fall into place
> after I merge the pull request!
>
> /Håkan
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/tidyui/Piranha/pull/141/files#r4954927>
> .
>