Skip to content

Apply BindingRedirects much faster - #414

Merged
forki merged 1 commit into
masterfrom
Redirect
Dec 4, 2014
Merged

Apply BindingRedirects much faster#414
forki merged 1 commit into
masterfrom
Redirect

Conversation

@forki

@forki forki commented Dec 3, 2014

Copy link
Copy Markdown
Member

Before: 138ms
Now: 51 ms

/cc @isaacabraham

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you use EndsWith? searchPattern was without wildcard, shouldn't it be simply x = "web.config" || x = "app.config"?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but It's the full path, right? Should be somethin like endwith DirectorySeparator + filename

forki added a commit that referenced this pull request Dec 4, 2014
Apply BindingRedirects much faster
@forki
forki merged commit 8fa70a1 into master Dec 4, 2014
@matthid
matthid deleted the Redirect branch May 6, 2017 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants