feat: Added datadoc and list access permissions for user groups

docs_website/docs/developer_guide/users_and_groups.mdx

@@ -17,7 +17,7 @@ A user or a group can have below properties
  - profile_img: Profile image url of a user or a group.
  - deleted: To indicate if this user has been deleted/deactivated or not.
  - is_group: To indicate it is a user group if it's true.
- - properties: Any addiontal properties are stored in this column. It is a freeform JSON object, you can basically add any properties inside it. By default all the properties inside `properties` are private, which are invisiable to end users.
+ - properties: Any additional properties are stored in this column. It is a freeform JSON object, you can basically add any properties inside it. By default all the properties inside `properties` are private, which are invisiable to end users.
     - public_info: Only properties stored inside `properties.public_info` are visible to end users.
       - description: [optional] Description of a user or a group.
 
@@ -31,3 +31,11 @@ For users
   - oauth/ldap authentication: they both support auto-creation of users.
 
 For user groups, they can only be sync'ed now and can not be created on UI. You can have a scheduled task for your organization to sync them from either ldap, metastore or any other system which contains user groups.
+
+
+## Group Permissions
+Querybook users can inherit permissions from groups to gain access to boards and datadocs alongside their explicit permissions. Likewise, user groups can also inherit permissions from other user groups. Whether a user or group is a member of two distinct groups or inherits permissions from multiple nested groups, the most permissive permissions are applied.
+
+For example, if a user is a member of two groups, one with read-only access and the other with write access, the user will have write access. If a user is a member of a group with read-only access and a group with no access, the user will have read-only access.
+
+User groups can be used to manage permissions for a large number of users. For example, you can create a group for all users in a team and assign permissions to the group. When a new user joins the team, you can add them to the group and they will inherit all of the permissions of the group. However, user groups cannot be assigned as the owners of a board or datadoc.

package.json

@@ -1,6 +1,6 @@
 {
     "name": "querybook",
-    "version": "3.39.0",
+    "version": "3.40.0",
     "description": "A Big Data Webapp",
     "private": true,
     "scripts": {

querybook/server/const/permissions.py

@@ -0,0 +1,6 @@
+from enum import Enum
+
+
+class BoardDataDocPermission(Enum):
+    READ = "read"
+    WRITE = "write"

querybook/server/datasources/board.py

@@ -11,7 +11,12 @@
 from const.datasources import RESOURCE_NOT_FOUND_STATUS_CODE
 from logic import board as logic
 from logic import user as user_logic
-from logic.board_permission import assert_can_read, assert_can_edit, assert_is_owner
+from logic.board_permission import (
+    assert_can_read,
+    assert_can_edit,
+    assert_is_owner,
+    assert_is_not_group,
+)
 from logic.query_execution import get_environments_by_execution_id
 
 from models.board import Board, BoardItem
@@ -293,6 +298,7 @@ def update_board_owner(board_id, next_owner_id, originator=None):
     with DBSession() as session:
         # Add previous owner as an editor to the doc
         assert_is_owner(board_id, session=session)
+        assert_is_not_group(next_owner_id, session=session)
         current_owner_editor = logic.create_board_editor(
             board_id=board_id,
             uid=current_user.id,

querybook/server/datasources/datadoc.py

@@ -25,7 +25,12 @@
     schedule as schedule_logic,
     user as user_logic,
 )
-from logic.datadoc_permission import assert_can_read, assert_can_write, assert_is_owner
+from logic.datadoc_permission import (
+    assert_can_read,
+    assert_can_write,
+    assert_is_owner,
+    assert_is_not_group,
+)
 from logic.query_execution import get_query_execution_by_id
 from logic.schedule import (
     run_and_log_scheduled_task,
@@ -622,6 +627,7 @@ def update_datadoc_owner(doc_id, next_owner_id, originator=None):
     with DBSession() as session:
         # Add previous owner as an editor to the doc
         assert_is_owner(doc_id, session=session)
+        assert_is_not_group(next_owner_id, session=session)
         current_owner_editor = logic.create_data_doc_editor(
             data_doc_id=doc_id,
             uid=current_user.id,

querybook/server/logic/board.py

@@ -1,12 +1,13 @@
 import datetime
 
-from sqlalchemy import or_
 from app.db import with_session
 from const.elasticsearch import ElasticsearchItem
 from models.board import Board, BoardItem, BoardEditor
 from models.access_request import AccessRequest
 from lib.sqlalchemy import update_model_fields
 from tasks.sync_elasticsearch import sync_elasticsearch
+from sqlalchemy import or_
+from logic.generic_permission import get_all_groups_and_group_members_with_access
 
 
 @with_session
@@ -232,13 +233,34 @@ def get_board_editor_by_id(id, session=None):
 
 @with_session
 def get_board_editors_by_board_id(board_id, session=None):
-    return session.query(BoardEditor).filter_by(board_id=board_id).all()
+    editors = get_all_groups_and_group_members_with_access(
+        doc_or_board_id=board_id,
+        editor_type=BoardEditor,
+        session=session,
+    )
+
+    return [
+        BoardEditor(
+            board_id=board_id,
+            id=id,
+            uid=uid,
+            read=read,
+            write=write,
+        )
+        for id, uid, read, write in editors
+    ]
 
 
 @with_session
 def create_board_editor(
     board_id, uid, read=False, write=False, commit=True, session=None
 ):
+    existing_editor = (
+        session.query(BoardEditor).filter_by(board_id=board_id, uid=uid).first()
+    )
+    if existing_editor is not None:
+        return existing_editor
+
     editor = BoardEditor(board_id=board_id, uid=uid, read=read, write=write)
 
     session.add(editor)

querybook/server/logic/board_permission.py

@@ -7,6 +7,9 @@
     ACCESS_RESTRICTED_STATUS_CODE,
     RESOURCE_NOT_FOUND_STATUS_CODE,
 )
+from const.permissions import BoardDataDocPermission
+
+from logic.generic_permission import user_has_permission
 
 
 class BoardDoesNotExist(Exception):
@@ -23,15 +26,10 @@ def user_can_edit(board_id, uid, session=None):
     if board.owner_uid == uid:
         return True
 
-    editor = (
-        session.query(BoardEditor)
-        .filter(BoardEditor.board_id == board_id)
-        .filter(BoardEditor.uid == uid)
-        .first()
+    return user_has_permission(
+        board_id, BoardDataDocPermission.WRITE, BoardEditor, uid, session=session
     )
 
-    return editor is not None and editor.write
-
 
 @with_session
 def user_can_read(board_id, uid, session=None):
@@ -40,21 +38,13 @@ def user_can_read(board_id, uid, session=None):
     if board is None:
         raise BoardDoesNotExist()
 
-    if board.public:
-        return True
-
-    if board.owner_uid == uid:
+    if board.public or board.owner_uid == uid:
         return True
 
-    editor = (
-        session.query(BoardEditor)
-        .filter(BoardEditor.board_id == board_id)
-        .filter(BoardEditor.uid == uid)
-        .first()
+    return user_has_permission(
+        board_id, BoardDataDocPermission.READ, BoardEditor, uid, session=session
     )
 
-    return editor is not None and (editor.read or editor.write)
-
 
 @with_session
 def assert_can_edit(board_id, session=None):
@@ -83,7 +73,7 @@ def assert_can_read(board_id, session=None):
 @with_session
 def assert_is_owner(board_id, session=None):
     try:
-        board = session.query(Board).filter(Board.id == board_id).first()
+        board = session.query(Board).get(board_id)
         if board is None:
             raise BoardDoesNotExist
         api_assert(
@@ -93,3 +83,15 @@ def assert_is_owner(board_id, session=None):
         )
     except BoardDoesNotExist:
         api_assert(False, "BOARD_DNE", RESOURCE_NOT_FOUND_STATUS_CODE)
+
+
+@with_session
+def assert_is_not_group(board_editor_id, session=None):
+    editor = session.query(BoardEditor).get(board_editor_id)
+    if editor is None:
+        api_assert(False, "EDITOR_DNE", RESOURCE_NOT_FOUND_STATUS_CODE)
+    api_assert(
+        editor.user.is_group is False or editor.user.is_group is None,
+        "Group cannot be assigned as owner",
+        ACCESS_RESTRICTED_STATUS_CODE,
+    )

querybook/server/logic/datadoc.py

@@ -9,6 +9,7 @@
 from lib.sqlalchemy import update_model_fields
 from lib.data_doc.data_cell import cell_types, sanitize_data_cell_meta
 from logic.query_execution import get_last_query_execution_from_cell
+from logic.generic_permission import get_all_groups_and_group_members_with_access
 from models.datadoc import (
     DataDoc,
     DataDocDataCell,
@@ -29,7 +30,6 @@
     sync_es_query_cells_by_datadoc_id,
 )
 
-
 """
     ----------------------------------------------------------------------------------------------------------
     DATA DOC
@@ -864,7 +864,23 @@ def get_data_doc_editor_by_id(id, session=None):
 
 @with_session
 def get_data_doc_editors_by_doc_id(data_doc_id, session=None):
-    return session.query(DataDocEditor).filter_by(data_doc_id=data_doc_id).all()
+    editors = get_all_groups_and_group_members_with_access(
+        doc_or_board_id=data_doc_id,
+        editor_type=DataDocEditor,
+        session=session,
+    )
+
+    return [
+        DataDocEditor(
+            # [0] is id, [1] is uid, [2] is read, [3] is write
+            data_doc_id=data_doc_id,
+            id=editor[0],
+            uid=editor[1],
+            read=editor[2],
+            write=editor[3],
+        )
+        for editor in editors
+    ]
 
 
 @with_session
@@ -876,6 +892,12 @@ def get_data_doc_writers_by_doc_id(doc_id, session=None):
 def create_data_doc_editor(
     data_doc_id, uid, read=False, write=False, commit=True, session=None
 ):
+    existing_editor = (
+        session.query(DataDocEditor).filter_by(data_doc_id=data_doc_id, uid=uid).first()
+    )
+    if existing_editor is not None:
+        return update_data_doc_editor(existing_editor.id, read, write, session=session)
+
     editor = DataDocEditor(data_doc_id=data_doc_id, uid=uid, read=read, write=write)
 
     session.add(editor)

querybook/server/logic/datadoc_permission.py

@@ -1,10 +1,16 @@
 from flask_login import current_user
-from sqlalchemy import and_
 
 from app.datasource import api_assert
 from app.db import with_session
 from models.datadoc import DataDoc, DataDocEditor
-from const.datasources import UNAUTHORIZED_STATUS_CODE, RESOURCE_NOT_FOUND_STATUS_CODE
+from const.datasources import (
+    UNAUTHORIZED_STATUS_CODE,
+    RESOURCE_NOT_FOUND_STATUS_CODE,
+    ACCESS_RESTRICTED_STATUS_CODE,
+)
+from const.permissions import BoardDataDocPermission
+
+from logic.generic_permission import user_has_permission
 
 
 class DocDoesNotExist(Exception):
@@ -13,34 +19,33 @@ class DocDoesNotExist(Exception):
 
 @with_session
 def user_can_write(doc_id, uid, session=None):
-    doc, editor = session.query(DataDoc, DataDocEditor).outerjoin(
-        DataDocEditor,
-        and_(DataDoc.id == DataDocEditor.data_doc_id, DataDocEditor.uid == uid),
-    ).filter(DataDoc.id == doc_id).first() or (None, None)
+    datadoc = session.query(DataDoc).get(doc_id)
 
-    if doc is None:
+    if datadoc is None:
         raise DocDoesNotExist()
 
-    if doc.owner_uid == uid:
+    if datadoc.owner_uid == uid:
         return True
 
-    return editor is not None and editor.write
+    return user_has_permission(
+        doc_id, BoardDataDocPermission.WRITE, DataDocEditor, uid, session=session
+    )
 
 
 @with_session
 def user_can_read(doc_id, uid, session=None):
-    doc, editor = session.query(DataDoc, DataDocEditor).outerjoin(
-        DataDocEditor,
-        and_(DataDoc.id == DataDocEditor.data_doc_id, DataDocEditor.uid == uid),
-    ).filter(DataDoc.id == doc_id).first() or (None, None)
+    # Check if the doc is public or if the user is the owner
+    datadoc = session.query(DataDoc).get(doc_id)
 
-    if doc is None:
+    if datadoc is None:
         raise DocDoesNotExist()
 
-    if doc.public or doc.owner_uid == uid:
+    if datadoc.public or datadoc.owner_uid == uid:
         return True
 
-    return editor is not None and (editor.read or editor.write)
+    return user_has_permission(
+        doc_id, BoardDataDocPermission.READ, DataDocEditor, uid, session=session
+    )
 
 
 @with_session
@@ -49,7 +54,7 @@ def assert_can_read(doc_id, session=None):
         api_assert(
             user_can_read(doc_id, uid=current_user.id, session=session),
             "CANNOT_READ_DATADOC",
-            UNAUTHORIZED_STATUS_CODE,
+            ACCESS_RESTRICTED_STATUS_CODE,
         )
     except DocDoesNotExist:
         api_assert(False, "DOC_DNE", RESOURCE_NOT_FOUND_STATUS_CODE)
@@ -70,7 +75,7 @@ def assert_can_write(doc_id, session=None):
 @with_session
 def assert_is_owner(doc_id, session=None):
     try:
-        doc = session.query(DataDoc).filter(DataDoc.id == doc_id).first()
+        doc = session.query(DataDoc).get(doc_id)
         if doc is None:
             raise DocDoesNotExist
         api_assert(
@@ -80,3 +85,15 @@ def assert_is_owner(doc_id, session=None):
         )
     except DocDoesNotExist:
         api_assert(False, "DOC_DNE", RESOURCE_NOT_FOUND_STATUS_CODE)
+
+
+@with_session
+def assert_is_not_group(editor_id, session=None):
+    editor = session.query(DataDocEditor).get(editor_id)
+    if editor is None:
+        api_assert(False, "EDITOR_DNE", RESOURCE_NOT_FOUND_STATUS_CODE)
+    api_assert(
+        editor.user.is_group is False or editor.user.is_group is None,
+        "Group cannot be assigned as owner",
+        ACCESS_RESTRICTED_STATUS_CODE,
+    )