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

Antiforgery filter is in wrong order relative to Authorize #5483

Closed
Eilon opened this issue Nov 1, 2016 · 6 comments
Closed

Antiforgery filter is in wrong order relative to Authorize #5483

Eilon opened this issue Nov 1, 2016 · 6 comments
Assignees
@ryanbrandenburg
Labels
3 - Done bug patch-approved
Milestone
1.0.2

Comments

@Eilon
Copy link
Member

Eilon commented Nov 1, 2016

See original issue w/ discussion and analysis here: aspnet/Security#1009

And pending PR here: #5470
@Eilon Eilon added this to the 1.0.2 milestone Nov 1, 2016
@Eilon
Copy link
Member Author

Eilon commented Nov 1, 2016

This patch is approved, please ensure it is merged into the correct branch and building as part of the patch train.

This was referenced Nov 1, 2016
Closed
Closed
@Eilon
Copy link
Member Author

Eilon commented Nov 1, 2016

@ryanbrandenburg please complete your PR, check into the patch build, then port to dev to ensure it goes into the 1.1.0 release. I logged #5484 to track the 1.1.0 work to make sure we don't forget.

@ryanbrandenburg
Copy link
Contributor

Fixed by #5470.

@ryanbrandenburg
Copy link
Contributor

Currently waiting on approval of #5470.

@ryanbrandenburg
Copy link
Contributor

Merged in f02f55c. Still waiting on the PR to add rel/1.0.2 to the patch train.

@ryanbrandenburg
Copy link
Contributor

rel/1.0.2 is in the patch train now, closing this.

@kichalla kichalla mentioned this issue Jan 10, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ryanbrandenburg ryanbrandenburg

Labels
3 - Done bug patch-approved
Projects
None yet
Milestone
1.0.2
Development

No branches or pull requests
2 participants
@Eilon @ryanbrandenburg