Skip to content
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.

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

⬆️ update to open-api-framework 0.9.1 #214

Merged
merged 1 commit into from
Jan 3, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
4 changes: 3 additions & 1 deletion docs/installation/configuration/env_config.rst
Original file line number Diff line number Diff line change
Expand Up @@ -91,10 +91,12 @@ Optional
* ``EMAIL_HOST_PASSWORD``: password to connect to the mail server. Defaults to: ``(empty string)``.
* ``EMAIL_USE_TLS``: whether to use TLS or not to connect to the mail server. Should be True if you're changing the ``EMAIL_PORT`` to 487. Defaults to: ``False``.
* ``DEFAULT_FROM_EMAIL``: The default email address from which emails are sent. Defaults to: ``[email protected]``.
* ``LOG_STDOUT``: whether to log to stdout or not. Defaults to: ``False``.
* ``LOG_STDOUT``: whether to log to stdout or not. Defaults to: ``True``.
* ``LOG_LEVEL``: control the verbosity of logging output. Available values are ``CRITICAL``, ``ERROR``, ``WARNING``, ``INFO`` and ``DEBUG``. Defaults to: ``WARNING``.
* ``LOG_QUERIES``: enable (query) logging at the database backend level. Note that you must also set ``DEBUG=1``, which should be done very sparingly!. Defaults to: ``False``.
* ``LOG_REQUESTS``: enable logging of the outgoing requests. Defaults to: ``False``.
* ``CELERY_LOGLEVEL``: control the verbosity of logging output for celery, independent of ``LOG_LEVEL``. Available values are ``CRITICAL``, ``ERROR``, ``WARNING``, ``INFO`` and ``DEBUG``. Defaults to: ``INFO``.
* ``SESSION_COOKIE_AGE``: For how long, in seconds, the session cookie will be valid. Defaults to: ``1209600``.
* ``SESSION_COOKIE_SAMESITE``: The value of the SameSite flag on the session cookie. This flag prevents the cookie from being sent in cross-site requests thus preventing CSRF attacks and making some methods of stealing session cookie impossible.Currently interferes with OIDC. Keep the value set at Lax if used. Defaults to: ``Lax``.
* ``CSRF_COOKIE_SAMESITE``: The value of the SameSite flag on the CSRF cookie. This flag prevents the cookie from being sent in cross-site requests. Defaults to: ``Strict``.
* ``ENVIRONMENT``: An identifier for the environment, displayed in the admin depending on the settings module used and included in the error monitoring (see ``SENTRY_DSN``). The default is set according to ``DJANGO_SETTINGS_MODULE``.
Expand Down
5 changes: 4 additions & 1 deletion requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ django==4.2.15
# django-relativedelta
# django-rest-framework-condition
# django-sendfile2
# django-sessionprofile
# django-setup-configuration
# django-simple-certmanager
# django-solo
Expand Down Expand Up @@ -153,6 +154,8 @@ django-rest-framework-condition==0.1.1
# via commonground-api-common
django-sendfile2==0.7.1
# via django-privates
django-sessionprofile==3.0.0
# via open-api-framework
django-setup-configuration==0.5.0
# via
# -r requirements/base.in
Expand Down Expand Up @@ -255,7 +258,7 @@ notifications-api-common[setup-configuration]==0.4.0
# via
# -r requirements/base.in
# commonground-api-common
open-api-framework==0.8.1
open-api-framework==0.9.1
# via -r requirements/base.in
orderedmultidict==1.0.1
# via furl
Expand Down
7 changes: 6 additions & 1 deletion requirements/ci.txt
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,7 @@ django==4.2.15
# django-relativedelta
# django-rest-framework-condition
# django-sendfile2
# django-sessionprofile
# django-setup-configuration
# django-simple-certmanager
# django-solo
Expand Down Expand Up @@ -242,6 +243,10 @@ django-sendfile2==0.7.1
# via
# -r requirements/base.txt
# django-privates
django-sessionprofile==3.0.0
# via
# -r requirements/base.txt
# open-api-framework
django-setup-configuration==0.5.0
# via
# -r requirements/base.txt
Expand Down Expand Up @@ -436,7 +441,7 @@ notifications-api-common[setup-configuration]==0.4.0
# -r requirements/base.txt
# commonground-api-common
# notifications-api-common
open-api-framework==0.8.1
open-api-framework==0.9.1
# via -r requirements/base.txt
orderedmultidict==1.0.1
# via
Expand Down
7 changes: 6 additions & 1 deletion requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ django==4.2.15
# django-relativedelta
# django-rest-framework-condition
# django-sendfile2
# django-sessionprofile
# django-setup-configuration
# django-simple-certmanager
# django-solo
Expand Down Expand Up @@ -249,6 +250,10 @@ django-sendfile2==0.7.1
# via
# -r requirements/base.txt
# django-privates
django-sessionprofile==3.0.0
# via
# -r requirements/base.txt
# open-api-framework
django-setup-configuration==0.5.0
# via
# -r requirements/base.txt
Expand Down Expand Up @@ -445,7 +450,7 @@ notifications-api-common[setup-configuration]==0.4.0
# -r requirements/base.txt
# commonground-api-common
# notifications-api-common
open-api-framework==0.8.1
open-api-framework==0.9.1
# via -r requirements/base.txt
orderedmultidict==1.0.1
# via
Expand Down
2 changes: 2 additions & 0 deletions src/nrc/accounts/tests/test_oidc.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
from functools import partial
from pathlib import Path
from unittest import skip
from unittest.mock import patch
from urllib.parse import urlparse

Expand Down Expand Up @@ -148,6 +149,7 @@ def test_happy_flow_existing_user(self):
self.assertEqual(staff_user.username, "admin")
self.assertEqual(staff_user.email, "[email protected]")

@skip("Session Refresh is disabled")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The middleware is removed in OAF

@mock_admin_oidc_config()
@patch(
"mozilla_django_oidc.middleware.SessionRefresh.is_refreshable_url",
Expand Down
4 changes: 4 additions & 0 deletions src/nrc/fixtures/default_admin_index.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,10 @@
[
"two_factor_webauthn",
"webauthndevice"
],
[
"sessionprofile",
"sessionprofile"
]
]
}
Expand Down
Loading