While updating to latest keto, I also updated to t...
# talk-keto
t
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
You want to update the keto-java-client documentation, did I understand that correctly? I can point you in the right direction, sec
s
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
Patrick is correct , the required annotations are the problem
s
would you be up for a fix?
t
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
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
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