Skip to content
This repository has been archived by the owner on Nov 22, 2018. It is now read-only.

Missing null check in CorsPolicyBuilder #51

Closed
wouterdevinck opened this issue Nov 20, 2015 · 1 comment
Closed

Missing null check in CorsPolicyBuilder #51

wouterdevinck opened this issue Nov 20, 2015 · 1 comment
Assignees
@kichalla
Labels
3 - Done bug
Milestone
1.0.0-rc2

Comments

@wouterdevinck
Copy link

The Combine method in in CorsPolicyBuilder is missing a null check on policy.PreFlightMaxAge: https://github.com/aspnet/CORS/blob/dev/src/Microsoft.AspNet.Cors/CorsPolicyBuilder.cs#L183
@Eilon Eilon added this to the 1.0.0-rc2 milestone Nov 20, 2015
@Eilon
Copy link
Member

Eilon commented Nov 20, 2015

@wouterdevinck nice catch!

@kichalla please make sure to add a test for this as well.

kichalla added a commit that referenced this issue Nov 23, 2015
@kichalla
[Fixes #51] Missing null check in CorsPolicyBuilder
33cf783
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kichalla kichalla

Labels
3 - Done bug
Projects
None yet
Milestone
1.0.0-rc2
Development

No branches or pull requests
3 participants
@Eilon @wouterdevinck @kichalla