Authorization Not Respected when returning ListGraphType of Authorized Type about authorization HOT 8 CLOSED

Once the build completes on the master branch I can push the nuget package to nuget.org. After I push it there it usually takes 10-15 minutes for nuget.org to index it.

from authorization.

And ... I just remembered I didn't bump the version. So doing that and running another master build.

from authorization.

I would expect that to fail. I think I know what the issue is:

fieldDef.ResolvedType should be getting the "root" GraphType here instead of just passing ResovledType, since that ResolvedType could be a NonNullGraphType<T> or ListGraphType<T>.

from authorization.

I was able to work around it for now by adding AuthorizeWith(...) to the field of the query directly. Would you consider this a fairly simple bug to fix? If so, I can take a crack at it.

from authorization.

@aaronjedwards It was fairly simple - pushed up #37 to fix it.

from authorization.

@joemcbride Awesome! Thanks for the quick turn around! I ask only because I don't have any insight into the matter, but how quickly can a simple bug fix like this get pushed through to the nuget package?

from authorization.

Its published, just need to wait for indexing.

https://www.nuget.org/packages/GraphQL.Authorization/2.1.29

from authorization.

@joemcbride Thanks again for the quick response. Ive got the latest package and it is working as intended!

from authorization.