Skip to content

Commit f33cb24

Browse files
authored
Removing excluded users (#171)
* Removing excluded users * Removing ObjectOrUserset which was only useful in the context of excluded users
1 parent 394820d commit f33cb24

File tree

5 files changed

+2138
-2216
lines changed

5 files changed

+2138
-2216
lines changed

docs/openapiv2/apidocs.swagger.json

Lines changed: 2 additions & 21 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

openfga/v1/openfga.proto

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,13 +49,6 @@ message User {
4949
}
5050
}
5151

52-
message ObjectOrUserset {
53-
oneof user {
54-
Object object = 1;
55-
UsersetUser userset = 2;
56-
}
57-
}
58-
5952
// Userset.
6053
//
6154
// A set or group of users, represented in the `<type>:<id>#<relation>` format

openfga/v1/openfga_service.proto

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -765,8 +765,9 @@ service OpenFGAService {
765765
"You may also provide a `context` object that will be used to evaluate the conditioned tuples in the system. It is strongly recommended to provide a value for all the input parameters of all the conditions, to ensure that all tuples be evaluated correctly.\n"
766766
"The response will contain the related users in an array in the \"users\" field of the response. These results may include specific objects, usersets \n"
767767
"or type-bound public access. Each of these types of results is encoded in its own type and not represented as a string."
768-
"In certain cases of negation via the `but not` operator, some results are marked as excluded from the main set of results. These exclusions \n"
769-
"are returned in the `excluded_users` property and should be handled appropriately at the point of implementation."
768+
"In cases where a type-bound public acces result is returned (e.g. `user:*`), it cannot be inferred that all subjects\n"
769+
"of that type have a relation to the object; it is possible that negations exist and checks should still be queried\n"
770+
"on individual subjects to ensure access to that document."
770771
"The number of users in the response array will be limited by the execution timeout specified in the flag OPENFGA_LIST_USERS_DEADLINE "
771772
"and by the upper bound specified in the flag OPENFGA_LIST_USERS_MAX_RESULTS, whichever is hit first.\n"
772773
"The returned users will not be sorted, and therefore two identical calls may yield different sets of users."
@@ -891,12 +892,6 @@ message ListUsersResponse {
891892
(google.api.field_behavior) = REQUIRED,
892893
(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {example: ""}
893894
];
894-
895-
repeated ObjectOrUserset excluded_users = 2 [
896-
json_name = "excluded_users",
897-
(google.api.field_behavior) = REQUIRED,
898-
(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {example: ""}
899-
];
900895
}
901896

902897
message StreamedListObjectsRequest {

0 commit comments

Comments
 (0)