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

Port ForbidAsync changes to 2.0.x #919

Closed
jkotalik opened this issue Aug 21, 2017 · 6 comments
Closed

Port ForbidAsync changes to 2.0.x #919

jkotalik opened this issue Aug 21, 2017 · 6 comments

Comments

@jkotalik
Copy link
Contributor

jkotalik commented Aug 21, 2017

See #917 and #918

@Eilon Eilon added this to the 2.0.1 milestone Aug 24, 2017
@Eilon
Copy link
Member

Eilon commented Sep 15, 2017

This patch bug is approved for the 2.0.x patch. Please send a PR to the feature/2.0.1 branch and get it reviewed and merged. When we have the rel/2.0.1 branches ready please port the commit to that branch.

@Eilon
Copy link
Member

Eilon commented Sep 15, 2017

@jkotalik / @HaoK / @Tratcher - Do we need to consider any quirks mode switch here? Is it even remotely possible that anyone would be relying on the 2.0.0-rtm behavior?

@Tratcher
Copy link
Member

No, Forbid already falls back to Challenge if not set.

@natemcmaster
Copy link
Contributor

Looks like this was fixed in #939, right @jkotalik ?

@jkotalik
Copy link
Contributor Author

Yes we did!

@Eilon
Copy link
Member

Eilon commented Oct 23, 2017

Hi, we have a public test feed that you can use to try out the ASP.NET/EF Core 2.0.3 patch!

To try out the pre-release patch, please refer to the following guide:

We are looking for feedback on this patch. We'd like to know if you have any issues with this patch by updating your apps and libraries to the latest packages and seeing if it fixes the issues you've had, or if it introduces any new issues. If you have any issues or questions, please reply on this issue to let us know as soon as possible.

Thanks,
Eilon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants