Allow passing a proc to authorize

[brendon]

I'm flirting with ActiveSupport::CurrentAttributes and have to currently add a bridging method to pass current_user into ActionPolicy:

authorize :user, through: :current_user

def current_user
  Current.user
end

Would you be open to allowing a proc?

authorize :user, through: -> { Current.user }

Of course I could just skip all that and use Current.user all over the policies. I'm not 100% sure how I feel about that yet.

Apart from the usual hate for Current is there any reason not to go this way? It would definitely make testing less isolated.

[pboling]

I think accepting a Proc makes sense because it can be molded to fit just about any need.

[brendon]

Sounds good, would you like me to do a PR?

[pboling]

I'm not a maintainer, just a fellow contributor, but of course I'd love to see it. :)

[brendon]

Oh lol :D Yea I'll wait to see what @palkan thinks :)

[palkan]

Sounds good to me! Will be happy to review and merge a PR :)

[brendon]

PR is up :)