Skip to content

events: add reason to the MODEL_UPDATED event for password hash updates #18068

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

Open
danidee10 wants to merge 13 commits into
base: main
Choose a base branch
from
Open

events: add reason to the MODEL_UPDATED event for password hash updates #18068

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
147cd34
feat: Add reason to the MODEL_UPDATED event for password hash updates
danidee10 Nov 11, 2025
edf3aaf
fix: Make the test_docker.sh script POSIX compliant
danidee10 Nov 11, 2025
1682c77
Merge remote-tracking branch 'origin/main' into feat/add-reason-passw…
danidee10 Nov 11, 2025
c173c38
fix: Fix bandit lint error
danidee10 Nov 11, 2025
ff8ff4f
chore: Add test for password hash without the request context
danidee10 Nov 11, 2025
7a3dcda
chore: simplify the Implementation with model_utils FieldTracker
danidee10 Nov 11, 2025
d3118c8
chore: Remove unnecessary test case
danidee10 Nov 11, 2025
d6afe94
Merge remote-tracking branch 'origin/main' into feat/add-reason-passw…
danidee10 Nov 20, 2025
391a3d3
feat: Distinguish between set-password and other hash upgrades
danidee10 Nov 20, 2025
db4087f
translate reason
BeryJu Jan 15, 2026
42693b4
feat: Ignore MODEL_UPDATED audit event with context manager
danidee10 Feb 7, 2026
1626674
Merge branch 'main' into feat/add-reason-password-hash-update
danidee10 Feb 7, 2026
36c9a2b
chore: Remove unused class variable
danidee10 Feb 7, 2026
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
Merge branch 'main' into feat/add-reason-password-hash-update 
Signed-off-by: Osaetin Daniel <osaetindaniel@gmail.com>
  • Loading branch information
@danidee10
danidee10 authored Feb 7, 2026
commit 16266749055c24310a33f24db32670e21efa5106
4 changes: 2 additions & 2 deletions scripts/test_docker.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ echo AUTHENTIK_SECRET_KEY="$(openssl rand -base64 60 | tr -d '\n')" >>lifecycle/
export COMPOSE_PROJECT_NAME="authentik-test-${AUTHENTIK_TAG}"

if [ -n "${BUILD:-}" ]; then

@danidee10 danidee10 Nov 11, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

When trying to test with docker. I discovered that this syntax wasn't POSIX compliant and didn't work on macos (with bash 3.2)

@melizeche melizeche Nov 11, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can confirm the same with zsh 5.9 (arm64-apple-darwin25.0)

echo AUTHENTIK_IMAGE="${AUTHENTIK_IMAGE}" >>.env
echo AUTHENTIK_TAG="${AUTHENTIK_TAG}" >>.env
echo AUTHENTIK_IMAGE="${AUTHENTIK_IMAGE}" >>lifecycle/container/.env
echo AUTHENTIK_TAG="${AUTHENTIK_TAG}" >>lifecycle/container/.env

# Ensure buildx is installed
docker buildx install
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.