Fix user deletion #26

Merged
dirk merged 1 commits from fix-user-deletion into main 1 day ago
jandd commented 2 days ago
Owner

Fixes #1544

The patch has originally been developed in https://github.com/CAcertOrg/cacert-devel/pull/37 and has been sent to critical-admin by Bernhard Fröhlich on Oct. 23rd 2022.

Message-ID: <b35e8587-6553-cc6c-76f2-def52d0aaf39@cacert.org>
From: Bernhard Fröhlich <bernhard@cacert.org>
To: "critical-admin@cacert.org" <critical-admin@cacert.org>
Subject: Patch request for bug #1544: Unable to delete users...
Date: Sun, 23 Oct 2022 14:27:25 +0200
Cc: CAcert-devel <cacert-devel@lists.cacert.org>

Hello critical admins,

we have a fix for <https://bugs.cacert.org/view.php?id=1544>.

The fix was officially reviewed by myself, and I successfully tested the patch
on test3.

A diff script is attached, it should only make a minimal change to the file
includes/notary.inc.php

Kind regards

Ted
Fixes #1544 The patch has originally been developed in https://github.com/CAcertOrg/cacert-devel/pull/37 and has been sent to critical-admin by Bernhard Fröhlich on Oct. 23rd 2022. ``` Message-ID: <b35e8587-6553-cc6c-76f2-def52d0aaf39@cacert.org> From: Bernhard Fröhlich <bernhard@cacert.org> To: "critical-admin@cacert.org" <critical-admin@cacert.org> Subject: Patch request for bug #1544: Unable to delete users... Date: Sun, 23 Oct 2022 14:27:25 +0200 Cc: CAcert-devel <cacert-devel@lists.cacert.org> Hello critical admins, we have a fix for <https://bugs.cacert.org/view.php?id=1544>. The fix was officially reviewed by myself, and I successfully tested the patch on test3. A diff script is attached, it should only make a minimal change to the file includes/notary.inc.php Kind regards Ted ```
jandd added 1 commit 2 days ago
jandd requested review from dirk 2 days ago
dirk approved these changes 2 days ago
dirk left a comment
Owner

change is as expected

change is as expected
dirk merged commit 7d0cb00de2 into main 1 day ago
jandd deleted branch fix-user-deletion 1 day ago

Reviewers

dirk approved these changes 2 days ago
The pull request has been merged as 7d0cb00de2.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cacert/cacert-webdb#26
Loading…
There is no content yet.