Skip to content

sqlalchemy-oso: handle get_checked_permissions() is None #1440

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cofinalsubnets merged 8 commits into from
Dec 10, 2021
Merged

sqlalchemy-oso: handle get_checked_permissions() is None #1440

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e1bb032
sqlalchemy no checked permissions bug fix
Dec 10, 2021
1b2e264
less specific allow rule
Dec 10, 2021
84c58ac
fmt
Dec 10, 2021
14f0a03
Merge branch 'main' into gwen/sqlalch-opt-map
Dec 10, 2021
93ca13c
changelog
Dec 10, 2021
f2389cb
Merge branch 'main' into gwen/sqlalch-opt-map
Dec 10, 2021
c1d937f
update black vn
Dec 10, 2021
6dac551
Merge branch 'main' into gwen/sqlalch-opt-map
Dec 10, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
update black vn
  • Loading branch information
Gwen Whelan committed Dec 10, 2021
commit c1d937fd61ce120e979755c5df6163c1123c7cf1
4 changes: 2 additions & 2 deletions languages/python/sqlalchemy-oso/requirements-dev.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
black==21.5b0
black~=21.12b
flake8==3.9.2
mypy==0.812
sqlalchemy-stubs==0.4
tox==3.23.1
tox==3.23.1
1 change: 0 additions & 1 deletion languages/python/sqlalchemy-oso/sqlalchemy_oso/session.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -316,7 +316,6 @@ def do_orm_execute(execute_state):
else:
logger.warning(f"Policy did not return filter for entity {entity}")


except ImportError:
from sqlalchemy.orm.query import Query

Expand Down
1 change: 0 additions & 1 deletion languages/python/sqlalchemy-oso/sqlalchemy_oso/sqlalchemy_utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,6 @@ def get_joinedload_entities(stmt):

return entities


except ImportError:
# This code should not be called for SQLAlchemy 1.4.
def all_entities_in_statement(statement):
Expand Down