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

Moving AllowAnonymous attribute from MVC #520

Merged
merged 1 commit into from
Oct 19, 2015
Merged

Conversation

ajaybhargavb
Copy link
Contributor

@dnfclas
Copy link

dnfclas commented Oct 15, 2015

Hi @ajaybhargavb, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;


namespace Microsoft.AspNet.Authorization
{
[AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is method right here? shouldn't this allow class also?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It allows class and method.

@rynowak
Copy link
Member

rynowak commented Oct 15, 2015

:shipit:

@ajaybhargavb ajaybhargavb merged commit 0f78135 into dev Oct 19, 2015
@Tratcher Tratcher deleted the allow-anonymous branch August 7, 2017 17:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants