Skip to content

Fix for NullReferenceException#21

Merged
Daniel15 merged 1 commit into
Daniel15:masterfrom
jFensch:master
Aug 6, 2013
Merged

Fix for NullReferenceException#21
Daniel15 merged 1 commit into
Daniel15:masterfrom
jFensch:master

Conversation

@jFensch

@jFensch jFensch commented Aug 6, 2013

Copy link
Copy Markdown
Contributor

I was running into a NullReferenceException during further testing.

@Daniel15

Daniel15 commented Aug 6, 2013

Copy link
Copy Markdown
Owner

Thanks! Do you know what routes caused this? Is it worth adding a unit test for?

Daniel15 added a commit that referenced this pull request Aug 6, 2013
Fix for NullReferenceException
@Daniel15 Daniel15 merged commit e597593 into Daniel15:master Aug 6, 2013
@jFensch

jFensch commented Aug 6, 2013

Copy link
Copy Markdown
Contributor Author

I believe it was the
route: routes.IgnoreRoute("{resource}.axd/{*pathInfo}");

On Tue, Aug 6, 2013 at 9:36 AM, Daniel Lo Nigro notifications@github.comwrote:

Thanks! Do you know what routes caused this? Is it worth adding a unit
test for?


Reply to this email directly or view it on GitHubhttps://github.com//pull/21#issuecomment-22178863
.

@Daniel15

Daniel15 commented Aug 6, 2013

Copy link
Copy Markdown
Owner

Thanks for reporting this, I've just added a unit test and released 1.1.3 which includes your fix.

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