https://www.ory.sh/ logo
t

thankful-traffic-31087

10/06/2021, 9:14 PM
While updating to latest keto, I also updated to the latest keto-java-client. But it seems that for a
GET /relation-tuples
the object and relation parameter are now required. But the documentation says that "List API: Display all Objects a User has Access to" is still possible. I could create a ticket or Pull Request for that, but I am unsure where to do that, due to being some kind of automatic process 🙂
m

magnificent-energy-493

10/07/2021, 8:01 AM
You want to update the keto-java-client documentation, did I understand that correctly? I can point you in the right direction, sec
s

steep-lamp-91158

10/07/2021, 8:43 AM
the spec is incorrect the problem is that here https://github.com/ory/keto/blob/cf7cddc62b6b010cdd31c911c56c3fc2d90cd8e8/internal/relationtuple/read_server.go#L50-L57 we reference that definition https://github.com/ory/keto/blob/cf7cddc62b6b010cdd31c911c56c3fc2d90cd8e8/internal/relationtuple/transact_server.go#L64-L110 but it should be a copy of that without the
// required: true
comments so the fix would be to copy the content of the
queryRelationTuple
into the
getRelationsParams
struct and remove all the required annotations
t

thankful-traffic-31087

10/07/2021, 1:35 PM
Patrick is correct , the required annotations are the problem
s

steep-lamp-91158

10/07/2021, 2:05 PM
would you be up for a fix?
t

thankful-traffic-31087

10/07/2021, 2:52 PM
Sure. As I understand it it would only require to change
<https://github.com/ory/keto-client-java/blob/master/api/openapi.yaml>
and than a new build would be required
s

steep-lamp-91158

10/07/2021, 3:08 PM
we auto-generate that file using
make sdk
from those code comments
so copy that type definition to one without the required annotations, and then run
make sdk
to update the spec
t

thankful-traffic-31087

10/07/2021, 5:00 PM
Hello, I added a PR, but it fails on cireleci . But i basically only changed some documentation .... but that seems to lead that auto generated code is now breaking a lot more than expected. due to some string vs *string
Got it fixed 😂 https://github.com/ory/keto/pull/760 is ready to go
1
2 Views