Merge pull request #7 from hishamco/dev

Adds CookiePath to SessionDefaults
dotnet · Mar 13, 2015 · c2cbd62 · c2cbd62
2 parents 20bcd63 + 177d1ff
commit c2cbd62
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/Microsoft.AspNet.Session/SessionDefaults.cs b/src/Microsoft.AspNet.Session/SessionDefaults.cs
@@ -8,5 +8,6 @@ namespace Microsoft.AspNet.Session
     public static class SessionDefaults
     {
         public static string CookieName = ".AspNet.Session";
+        public static string CookiePath = "/";
     }
 }
diff --git a/src/Microsoft.AspNet.Session/SessionMiddleware.cs b/src/Microsoft.AspNet.Session/SessionMiddleware.cs
@@ -126,7 +126,7 @@ private void SetCookie()
                 {
                     Domain = _options.CookieDomain,
                     HttpOnly = _options.CookieHttpOnly,
-                    Path = _options.CookiePath ?? "/",
+                    Path = _options.CookiePath ?? SessionDefaults.CookiePath,
                 };
 
                 _context.Response.Cookies.Append(_options.CookieName, _sessionKey, cookieOptions);

diff --git a/src/Microsoft.AspNet.Session/SessionOptions.cs b/src/Microsoft.AspNet.Session/SessionOptions.cs
@@ -20,7 +20,7 @@ public class SessionOptions
         /// <summary>
         /// Determines the path used to create the cookie. The default value is "/" for highest browser compatibility.
         /// </summary>
-        public string CookiePath { get; set; } = "/";
+        public string CookiePath { get; set; } = SessionDefaults.CookiePath;
 
         /// <summary>
         /// Determines if the browser should allow the cookie to be accessed by client-side JavaScript. The