authzed / authzed-rb

Official SpiceDB client library for Ruby
https://authzed.com/docs
Apache License 2.0
17 stars 11 forks source link

Please Add `permissions_service.check_bulk_permissions` #131

Closed MichaelCordingley closed 4 days ago

MichaelCordingley commented 5 days ago

The experimental service is in this client as of Nov. 2023, but not the final version that shipped as part of SpiceDB 1.30.0 in March 2024.

tstirrat15 commented 5 days ago

@MichaelCordingley are you referring to this? It should be available.

MichaelCordingley commented 5 days ago

No. That's the message. There does not appear to be a way to send that message to the server.

tstirrat15 commented 5 days ago

Got it. Lemme dig. Thanks for letting us know!

tstirrat15 commented 4 days ago

@MichaelCordingley can you check out the associated PR? It appears that the check_bulk_permission service is available and usable in the client. Where were you seeing that it wasn't?

MichaelCordingley commented 4 days ago

I'm using the client version 0.7.1 and ran this code to verify its non-existence:

[3] pry(main)> authzed_client.permissions_service.check_bulk_permissions
NoMethodError: undefined method `check_bulk_permissions' for #<Authzed::Api::V1::PermissionsService::Stub:0x000000012fd52818 @ch=#<GRPC::Core::Channel:0x0000000150a56f80>, @host="localhost:9090", @propagate_mask=nil, @timeout=1969-12-31 18:59:59 -0500, @interceptors=#<GRPC::InterceptorRegistry:0x0000000150a55e00 @interceptors=[#<Authzed::GrpcUtil::BearerToken:0x0000000150a5cef8 @token="presharedkey">]>>
from (pry):3:in `__pry__'

I don't see reference to this in the release notes for versions after 0.7.1, but can rerun this on the latest release ~tomorrow~.

MichaelCordingley commented 4 days ago

Confirmed present on 0.11.0:

[2] pry(main)> authzed_client.permissions_service.check_bulk_permissions
ArgumentError: wrong number of arguments (given 0, expected 1..2)
from /Users/michael.cordingley/.asdf/installs/ruby/3.2.5/lib/ruby/gems/3.2.0/gems/grpc-1.66.0-arm64-darwin/src/ruby/lib/grpc/generic/service.rb:169:in `block (3 levels) in rpc_stub_class'
tstirrat15 commented 4 days ago

It happened in #91, which was present in this release. It looks like your version is just before that one.

MichaelCordingley commented 4 days ago

Appreciate the assist!