Verified Commit ff92e95f authored by Leonidas Spyropoulos's avatar Leonidas Spyropoulos
Browse files

fix: delete associated ssh public keys with account deletion

Signed-off-by: Leonidas Spyropoulos's avatarLeonidas Spyropoulos <>
parent bce5b81a
Pipeline #39783 passed with stages
in 2 minutes and 10 seconds
......@@ -13,7 +13,7 @@ class SSHPubKey(Base):
User = relationship(
backref=backref("ssh_pub_keys", lazy="dynamic"),
backref=backref("ssh_pub_keys", lazy="dynamic", cascade="all, delete"),
......@@ -2032,6 +2032,37 @@ def test_account_delete_self(client: TestClient, user: User):
assert record is None
def test_account_delete_self_with_ssh_public_key(client: TestClient, user: User):
username = user.Username
with db.begin():
SSHPubKey, User=user, Fingerprint="testFingerprint", PubKey="testPubKey"
# Confirm that we can view our own account deletion page
cookies = {"AURSID": user.login(Request(), "testPassword")}
endpoint = f"/account/{username}/delete"
with client as request:
resp = request.get(endpoint, cookies=cookies)
assert resp.status_code == HTTPStatus.OK
# Supply everything correctly and delete ourselves
with client as request:
resp =
data={"passwd": "testPassword", "confirm": True},
assert resp.status_code == HTTPStatus.SEE_OTHER
# Check that our User record no longer exists in the database
user_record = db.query(User).filter(User.Username == username).first()
assert user_record is None
sshpubkey_record = db.query(SSHPubKey).filter(SSHPubKey.User == user).first()
assert sshpubkey_record is None
def test_account_delete_as_tu(client: TestClient, tu_user: User):
with db.begin():
user = create_user("user2")
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment