search

dotnet-websharper / owin

Wrappers for hosting WebSharper sitelets and remoting components in OWIN projects
Apache License 2.0
10 stars 9 forks source link

Added the current Owin context to IContext's enviroment dictionary. #9

Closed cata closed 9 years ago

cata commented 9 years ago

Fixes #7

Also, prevented OwinCookieUserSession from overwriting Owin context's principal. This fixes a class of bugs where the principal set by Owin authentication middleware was over written by OwinCookieUserSession

Note that this must be merged at the same time as websharper #388, due to changes to Web.IContext

Tarmil commented 9 years ago

This looks good, thanks!

cata commented 9 years ago

Thanks for merging!

It would be great if you could update the nuget.org packages.

cata commented 9 years ago

The reason I'm asking is because I do not have access to the sources of Websharper.Highcharts (which I am using), and it needs to be recompiled in order for metadata deserialization to work with the modified Websharper.

Or, if possible, please put the Websharper.Highcharts sources on Github - that'd be awesome, as I have stumbled upon some issues using it (such as the lack of per-series configuration options) that I could try to fix and submit PRs for.

Thanks!