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

Copy link
Copy Markdown
Contributor

I was running into a NullReferenceException during further testing.

@Daniel15

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:masterAug 6, 2013
@jFensch

Copy link
Copy Markdown
ContributorAuthor

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

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 freeto 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

@jFensch@Daniel15