Skip to content
Snippets Groups Projects
Commit c9ddc977 authored by Simon Künzel's avatar Simon Künzel
Browse files

Fix fsmpi authentication

parent a138463f
No related branches found
No related tags found
No related merge requests found
Pipeline #7659 failed
Pipeline: backend

#7660

    ...@@ -223,6 +223,7 @@ def authenticate_fsmpi(username: str, password: str) -> {}: ...@@ -223,6 +223,7 @@ def authenticate_fsmpi(username: str, password: str) -> {}:
    if user_db is None: if user_db is None:
    raise ApiClientException(ERROR_AUTHENTICATION_NOT_AVAILABLE( raise ApiClientException(ERROR_AUTHENTICATION_NOT_AVAILABLE(
    "Site is read-only and we can not create a new account for you in the database")) "Site is read-only and we can not create a new account for you in the database"))
    api_session.set_user(user_db)
    else: else:
    def _trans(session: SessionDb): def _trans(session: SessionDb):
    user_db = session.scalar(User.sudo_select().where(User.handle == user_handle)) user_db = session.scalar(User.sudo_select().where(User.handle == user_handle))
    ...@@ -241,13 +242,10 @@ def authenticate_fsmpi(username: str, password: str) -> {}: ...@@ -241,13 +242,10 @@ def authenticate_fsmpi(username: str, password: str) -> {}:
    user_db.full_name = full_name user_db.full_name = full_name
    user_db.display_name = display_name user_db.display_name = display_name
    user_db.email = email user_db.email = email
    api_session.set_user(user_db)
    session.commit() database.execute_write_transaction_and_commit(_trans)
    session.expunge_all()
    return user_db
    user_db = database.execute_write_transaction(_trans)
    api_session.set_user(user_db)
    api_session.csrf_token = "".join( api_session.csrf_token = "".join(
    secrets.choice(string.ascii_letters + string.digits) for _ in range(64)) secrets.choice(string.ascii_letters + string.digits) for _ in range(64))
    api_session.permanent = True api_session.permanent = True
    ......
    ...@@ -1502,7 +1502,7 @@ class ObjectModificationsTest(ApiTest): ...@@ -1502,7 +1502,7 @@ class ObjectModificationsTest(ApiTest):
    f"/object_management/course/{id}/configuration", f"/object_management/course/{id}/configuration",
    use_moderator_login=True, use_moderator_login=True,
    )[1] )[1]
    self.assert_field_has_value(config, "responsible_users_ids", [42, 43]) self.assertEqual(set(self.get_field_value(config, "responsible_users_ids")), {42, 43})
    self.do_json_request( self.do_json_request(
    "GET", "GET",
    f"/course/{id}", f"/course/{id}",
    ......
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Please register or to comment