Assign normalized target to new variable.

Cesar Blum Silveira · Cesar Blum Silveira · commit 7f2aaac0ea63 · 2016-05-31T15:09:03.000-07:00
diff --git a/src/Microsoft.AspNetCore.Server.Kestrel/Http/Frame.cs b/src/Microsoft.AspNetCore.Server.Kestrel/Http/Frame.cs
@@ -927,7 +927,7 @@ protected RequestLineStatus TakeStartLine(SocketInput input)
                     }
                 }
 
-                requestUrlPath = PathNormalizer.RemoveDotSegments(requestUrlPath);
+                var normalizedTarget = PathNormalizer.RemoveDotSegments(requestUrlPath);
 
                 consumed = scan;
                 Method = method;
@@ -936,14 +936,14 @@ protected RequestLineStatus TakeStartLine(SocketInput input)
                 HttpVersion = httpVersion;
 
                 bool caseMatches;
-                if (RequestUrlStartsWithPathBase(requestUrlPath, out caseMatches))
+                if (RequestUrlStartsWithPathBase(normalizedTarget, out caseMatches))
                 {
-                    PathBase = caseMatches ? _pathBase : requestUrlPath.Substring(0, _pathBase.Length);
-                    Path = requestUrlPath.Substring(_pathBase.Length);
+                    PathBase = caseMatches ? _pathBase : normalizedTarget.Substring(0, _pathBase.Length);
+                    Path = normalizedTarget.Substring(_pathBase.Length);
                 }
-                else if (rawTarget[0] == '/') // check rawTarget since requestUrlPath can be "" or "/" after dot segment removal
+                else if (rawTarget[0] == '/') // check rawTarget since normalizedTarget can be "" or "/" after dot segment removal
                 {
-                    Path = requestUrlPath;
+                    Path = normalizedTarget;
                 }
                 else
                 {

Original file line number	Diff line number	Diff line change
`@@ -927,7 +927,7 @@ protected RequestLineStatus TakeStartLine(SocketInput input)`
`927`	`927`	`}`
`928`	`928`	`}`
`929`	`929`
`930`		`- requestUrlPath = PathNormalizer.RemoveDotSegments(requestUrlPath);`
	`930`	`+ var normalizedTarget = PathNormalizer.RemoveDotSegments(requestUrlPath);`
`931`	`931`
`932`	`932`	`consumed = scan;`
`933`	`933`	`Method = method;`
`@@ -936,14 +936,14 @@ protected RequestLineStatus TakeStartLine(SocketInput input)`
`936`	`936`	`HttpVersion = httpVersion;`
`937`	`937`
`938`	`938`	`bool caseMatches;`
`939`		`- if (RequestUrlStartsWithPathBase(requestUrlPath, out caseMatches))`
	`939`	`+ if (RequestUrlStartsWithPathBase(normalizedTarget, out caseMatches))`
`940`	`940`	`{`
`941`		`- PathBase = caseMatches ? _pathBase : requestUrlPath.Substring(0, _pathBase.Length);`
`942`		`- Path = requestUrlPath.Substring(_pathBase.Length);`
	`941`	`+ PathBase = caseMatches ? _pathBase : normalizedTarget.Substring(0, _pathBase.Length);`
	`942`	`+ Path = normalizedTarget.Substring(_pathBase.Length);`
`943`	`943`	`}`
`944`		`- else if (rawTarget[0] == '/') // check rawTarget since requestUrlPath can be "" or "/" after dot segment removal`
	`944`	`+ else if (rawTarget[0] == '/') // check rawTarget since normalizedTarget can be "" or "/" after dot segment removal`
`945`	`945`	`{`
`946`		`- Path = requestUrlPath;`
	`946`	`+ Path = normalizedTarget;`
`947`	`947`	`}`
`948`	`948`	`else`
`949`	`949`	`{`