Compare commits
175 Commits
dependabot
...
policies/o
Author | SHA1 | Date | |
---|---|---|---|
6e1cf8a23c | |||
fde6120e67 | |||
dabd812071 | |||
db92da4cb8 | |||
81c23fff98 | |||
54b5774a15 | |||
ba4650a088 | |||
b7d2c5188b | |||
dd8e71df7d | |||
baa4deda99 | |||
b7417e77c7 | |||
a01bb551d0 | |||
9a03bdeaf1 | |||
6b530ff764 | |||
e77a3241f0 | |||
a718ff2617 | |||
969fa82b7f | |||
bb47a70310 | |||
a306cecb73 | |||
760879c3db | |||
ef5d3580e8 | |||
d14b480926 | |||
d9c79558b1 | |||
ed20d1b6aa | |||
f03ee47bb3 | |||
396366a99a | |||
13d2df3bf6 | |||
d65b8ae029 | |||
296031c5df | |||
452639d6d2 | |||
465ccb7ab9 | |||
fdce812ddc | |||
005da84dbe | |||
b098971718 | |||
147bfa3f97 | |||
fc5f91ea29 | |||
e29961b088 | |||
52ca70d6bb | |||
42cb9cb531 | |||
837cd3bcb0 | |||
53a9c147cd | |||
d7f166f260 | |||
8ce9343457 | |||
6af27d0c90 | |||
6fd48ccf9b | |||
5567967848 | |||
090a377c78 | |||
3e7bda87ea | |||
f22a539b50 | |||
54811b2b05 | |||
35263f71ee | |||
f0bc809389 | |||
75b45312ee | |||
e4eeb43f8a | |||
04850e5c84 | |||
fbae9f2f34 | |||
3c966d9252 | |||
9f1cef18b2 | |||
aae20dc399 | |||
4dc43b788a | |||
a3b40a97ef | |||
852106f02f | |||
7a34428aff | |||
c1b6a681a0 | |||
7a8c2e7ad9 | |||
a57381ca4a | |||
154dde9a9a | |||
a15365a9f1 | |||
10f11cbc31 | |||
caec23d52a | |||
7e1781ed76 | |||
0cfdbbbec6 | |||
8a1b7cb166 | |||
f367a84676 | |||
32d6b03a3c | |||
08027bf0ad | |||
8c02b25677 | |||
160f137707 | |||
52c35fab06 | |||
69a07c1c88 | |||
691a0d66ee | |||
3f4328bf2a | |||
b945552b7c | |||
5347b85c9f | |||
fb2401cf9e | |||
b161315811 | |||
0fa2267b86 | |||
4bbdddb876 | |||
bca9c0965e | |||
dd58b5044e | |||
c4f081cb68 | |||
59aad31459 | |||
de9db3cb83 | |||
24eb5fcda9 | |||
556ae6a5cb | |||
a479d9c1d8 | |||
b8bb969ee7 | |||
7d361e4734 | |||
dc7c7686a3 | |||
94b4977397 | |||
7f822e1cb7 | |||
fb3ec1f38b | |||
87505517ee | |||
4c5fe84f92 | |||
5faa224c81 | |||
736da3abef | |||
52d90f8d3b | |||
7b812de977 | |||
a4bd2cc263 | |||
14038ba8d2 | |||
eaff59b6b0 | |||
cb702ca07a | |||
cb0bfb0dad | |||
bf46d5c916 | |||
59e686c8b9 | |||
9e736f2838 | |||
c2dd3d9c1b | |||
42302d3187 | |||
20ccabf3ec | |||
8f939fa577 | |||
2519bcef89 | |||
3e3615a859 | |||
79e82c8dc9 | |||
ccd4432e1f | |||
b3137f5307 | |||
2591ed9840 | |||
b3e89ef570 | |||
45b48c5cd6 | |||
1eefd834fc | |||
4cc6ed97c5 | |||
bb55d9b3de | |||
3972afb865 | |||
04a013cc1b | |||
fb396f7737 | |||
cf120ff3ff | |||
3e4923d52e | |||
01793088f0 | |||
e2bf2ec2cc | |||
4dfbe28709 | |||
b2021a7191 | |||
81e5fb0c18 | |||
a2a2d940a8 | |||
c034930219 | |||
da3dc51d87 | |||
d217a39513 | |||
7729a9317c | |||
be5f5dd3f0 | |||
bed8d5da4b | |||
4f70f84e80 | |||
97b8551866 | |||
9a0b67e700 | |||
97e4c89cec | |||
65aedde8f7 | |||
17450f23bf | |||
ab3ad6b7fd | |||
45bc3cbd41 | |||
9c1bcac6af | |||
0a133265c5 | |||
57f25a97c9 | |||
8f32242787 | |||
c4bb19051d | |||
10f4fae711 | |||
2d9eab3f60 | |||
fa66195619 | |||
134eb126b6 | |||
f5a6136a58 | |||
1a82dfcd61 | |||
61fc1dc1fb | |||
1f921cc18e | |||
2f94ee3f1f | |||
154fba12e0 | |||
0d18c1d797 | |||
e905dd52d8 | |||
245126a1c3 | |||
15d84d30ba |
@ -1,16 +1,16 @@
|
|||||||
[bumpversion]
|
[bumpversion]
|
||||||
current_version = 2025.4.1
|
current_version = 2025.6.0
|
||||||
tag = True
|
tag = True
|
||||||
commit = True
|
commit = True
|
||||||
parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(?:-(?P<rc_t>[a-zA-Z-]+)(?P<rc_n>[1-9]\\d*))?
|
parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(?:-(?P<rc_t>[a-zA-Z-]+)(?P<rc_n>[1-9]\\d*))?
|
||||||
serialize =
|
serialize =
|
||||||
{major}.{minor}.{patch}-{rc_t}{rc_n}
|
{major}.{minor}.{patch}-{rc_t}{rc_n}
|
||||||
{major}.{minor}.{patch}
|
{major}.{minor}.{patch}
|
||||||
message = release: {new_version}
|
message = release: {new_version}
|
||||||
tag_name = version/{new_version}
|
tag_name = version/{new_version}
|
||||||
|
|
||||||
[bumpversion:part:rc_t]
|
[bumpversion:part:rc_t]
|
||||||
values =
|
values =
|
||||||
rc
|
rc
|
||||||
final
|
final
|
||||||
optional_value = final
|
optional_value = final
|
||||||
|
@ -5,6 +5,7 @@ dist/**
|
|||||||
build/**
|
build/**
|
||||||
build_docs/**
|
build_docs/**
|
||||||
*Dockerfile
|
*Dockerfile
|
||||||
|
**/*Dockerfile
|
||||||
blueprints/local
|
blueprints/local
|
||||||
.git
|
.git
|
||||||
!gen-ts-api/node_modules
|
!gen-ts-api/node_modules
|
||||||
|
2
.github/actions/setup/action.yml
vendored
2
.github/actions/setup/action.yml
vendored
@ -36,7 +36,7 @@ runs:
|
|||||||
with:
|
with:
|
||||||
go-version-file: "go.mod"
|
go-version-file: "go.mod"
|
||||||
- name: Setup docker cache
|
- name: Setup docker cache
|
||||||
uses: ScribeMD/docker-cache@0.5.0
|
uses: AndreKurait/docker-cache@0fe76702a40db986d9663c24954fc14c6a6031b7
|
||||||
with:
|
with:
|
||||||
key: docker-images-${{ runner.os }}-${{ hashFiles('.github/actions/setup/docker-compose.yml', 'Makefile') }}-${{ inputs.postgresql_version }}
|
key: docker-images-${{ runner.os }}-${{ hashFiles('.github/actions/setup/docker-compose.yml', 'Makefile') }}-${{ inputs.postgresql_version }}
|
||||||
- name: Setup dependencies
|
- name: Setup dependencies
|
||||||
|
62
.github/workflows/ci-website.yml
vendored
62
.github/workflows/ci-website.yml
vendored
@ -41,32 +41,60 @@ jobs:
|
|||||||
- name: test
|
- name: test
|
||||||
working-directory: website/
|
working-directory: website/
|
||||||
run: npm test
|
run: npm test
|
||||||
build:
|
build-container:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
name: ${{ matrix.job }}
|
permissions:
|
||||||
strategy:
|
# Needed to upload container images to ghcr.io
|
||||||
fail-fast: false
|
packages: write
|
||||||
matrix:
|
# Needed for attestation
|
||||||
job:
|
id-token: write
|
||||||
- build
|
attestations: write
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
- uses: actions/setup-node@v4
|
|
||||||
with:
|
with:
|
||||||
node-version-file: website/package.json
|
ref: ${{ github.event.pull_request.head.sha }}
|
||||||
cache: "npm"
|
- name: Set up QEMU
|
||||||
cache-dependency-path: website/package-lock.json
|
uses: docker/setup-qemu-action@v3.6.0
|
||||||
- working-directory: website/
|
- name: Set up Docker Buildx
|
||||||
run: npm ci
|
uses: docker/setup-buildx-action@v3
|
||||||
- name: build
|
- name: prepare variables
|
||||||
working-directory: website/
|
uses: ./.github/actions/docker-push-variables
|
||||||
run: npm run ${{ matrix.job }}
|
id: ev
|
||||||
|
env:
|
||||||
|
DOCKER_USERNAME: ${{ secrets.DOCKER_USERNAME }}
|
||||||
|
with:
|
||||||
|
image-name: ghcr.io/goauthentik/dev-docs
|
||||||
|
- name: Login to Container Registry
|
||||||
|
if: ${{ steps.ev.outputs.shouldPush == 'true' }}
|
||||||
|
uses: docker/login-action@v3
|
||||||
|
with:
|
||||||
|
registry: ghcr.io
|
||||||
|
username: ${{ github.repository_owner }}
|
||||||
|
password: ${{ secrets.GITHUB_TOKEN }}
|
||||||
|
- name: Build Docker Image
|
||||||
|
id: push
|
||||||
|
uses: docker/build-push-action@v6
|
||||||
|
with:
|
||||||
|
tags: ${{ steps.ev.outputs.imageTags }}
|
||||||
|
file: website/Dockerfile
|
||||||
|
push: ${{ steps.ev.outputs.shouldPush == 'true' }}
|
||||||
|
platforms: linux/amd64,linux/arm64
|
||||||
|
context: .
|
||||||
|
cache-from: type=registry,ref=ghcr.io/goauthentik/dev-docs:buildcache
|
||||||
|
cache-to: ${{ steps.ev.outputs.shouldPush == 'true' && 'type=registry,ref=ghcr.io/goauthentik/dev-docs:buildcache,mode=max' || '' }}
|
||||||
|
- uses: actions/attest-build-provenance@v2
|
||||||
|
id: attest
|
||||||
|
if: ${{ steps.ev.outputs.shouldPush == 'true' }}
|
||||||
|
with:
|
||||||
|
subject-name: ${{ steps.ev.outputs.attestImageNames }}
|
||||||
|
subject-digest: ${{ steps.push.outputs.digest }}
|
||||||
|
push-to-registry: true
|
||||||
ci-website-mark:
|
ci-website-mark:
|
||||||
if: always()
|
if: always()
|
||||||
needs:
|
needs:
|
||||||
- lint
|
- lint
|
||||||
- test
|
- test
|
||||||
- build
|
- build-container
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: re-actors/alls-green@release/v1
|
- uses: re-actors/alls-green@release/v1
|
||||||
|
45
.github/workflows/release-publish.yml
vendored
45
.github/workflows/release-publish.yml
vendored
@ -20,6 +20,49 @@ jobs:
|
|||||||
release: true
|
release: true
|
||||||
registry_dockerhub: true
|
registry_dockerhub: true
|
||||||
registry_ghcr: true
|
registry_ghcr: true
|
||||||
|
build-docs:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
permissions:
|
||||||
|
# Needed to upload container images to ghcr.io
|
||||||
|
packages: write
|
||||||
|
# Needed for attestation
|
||||||
|
id-token: write
|
||||||
|
attestations: write
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
- name: Set up QEMU
|
||||||
|
uses: docker/setup-qemu-action@v3.6.0
|
||||||
|
- name: Set up Docker Buildx
|
||||||
|
uses: docker/setup-buildx-action@v3
|
||||||
|
- name: prepare variables
|
||||||
|
uses: ./.github/actions/docker-push-variables
|
||||||
|
id: ev
|
||||||
|
env:
|
||||||
|
DOCKER_USERNAME: ${{ secrets.DOCKER_USERNAME }}
|
||||||
|
with:
|
||||||
|
image-name: ghcr.io/goauthentik/docs
|
||||||
|
- name: Login to GitHub Container Registry
|
||||||
|
uses: docker/login-action@v3
|
||||||
|
with:
|
||||||
|
registry: ghcr.io
|
||||||
|
username: ${{ github.repository_owner }}
|
||||||
|
password: ${{ secrets.GITHUB_TOKEN }}
|
||||||
|
- name: Build Docker Image
|
||||||
|
id: push
|
||||||
|
uses: docker/build-push-action@v6
|
||||||
|
with:
|
||||||
|
tags: ${{ steps.ev.outputs.imageTags }}
|
||||||
|
file: website/Dockerfile
|
||||||
|
push: true
|
||||||
|
platforms: linux/amd64,linux/arm64
|
||||||
|
context: .
|
||||||
|
- uses: actions/attest-build-provenance@v2
|
||||||
|
id: attest
|
||||||
|
if: true
|
||||||
|
with:
|
||||||
|
subject-name: ${{ steps.ev.outputs.attestImageNames }}
|
||||||
|
subject-digest: ${{ steps.push.outputs.digest }}
|
||||||
|
push-to-registry: true
|
||||||
build-outpost:
|
build-outpost:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
permissions:
|
permissions:
|
||||||
@ -193,6 +236,6 @@ jobs:
|
|||||||
SENTRY_ORG: authentik-security-inc
|
SENTRY_ORG: authentik-security-inc
|
||||||
SENTRY_PROJECT: authentik
|
SENTRY_PROJECT: authentik
|
||||||
with:
|
with:
|
||||||
version: authentik@${{ steps.ev.outputs.version }}
|
release: authentik@${{ steps.ev.outputs.version }}
|
||||||
sourcemaps: "./web/dist"
|
sourcemaps: "./web/dist"
|
||||||
url_prefix: "~/static/dist"
|
url_prefix: "~/static/dist"
|
||||||
|
48
Dockerfile
48
Dockerfile
@ -1,26 +1,7 @@
|
|||||||
# syntax=docker/dockerfile:1
|
# syntax=docker/dockerfile:1
|
||||||
|
|
||||||
# Stage 1: Build website
|
# Stage 1: Build webui
|
||||||
FROM --platform=${BUILDPLATFORM} docker.io/library/node:24 AS website-builder
|
FROM --platform=${BUILDPLATFORM} docker.io/library/node:24-slim AS node-builder
|
||||||
|
|
||||||
ENV NODE_ENV=production
|
|
||||||
|
|
||||||
WORKDIR /work/website
|
|
||||||
|
|
||||||
RUN --mount=type=bind,target=/work/website/package.json,src=./website/package.json \
|
|
||||||
--mount=type=bind,target=/work/website/package-lock.json,src=./website/package-lock.json \
|
|
||||||
--mount=type=cache,id=npm-website,sharing=shared,target=/root/.npm \
|
|
||||||
npm ci --include=dev
|
|
||||||
|
|
||||||
COPY ./website /work/website/
|
|
||||||
COPY ./blueprints /work/blueprints/
|
|
||||||
COPY ./schema.yml /work/
|
|
||||||
COPY ./SECURITY.md /work/
|
|
||||||
|
|
||||||
RUN npm run build-bundled
|
|
||||||
|
|
||||||
# Stage 2: Build webui
|
|
||||||
FROM --platform=${BUILDPLATFORM} docker.io/library/node:24 AS web-builder
|
|
||||||
|
|
||||||
ARG GIT_BUILD_HASH
|
ARG GIT_BUILD_HASH
|
||||||
ENV GIT_BUILD_HASH=$GIT_BUILD_HASH
|
ENV GIT_BUILD_HASH=$GIT_BUILD_HASH
|
||||||
@ -32,7 +13,7 @@ RUN --mount=type=bind,target=/work/web/package.json,src=./web/package.json \
|
|||||||
--mount=type=bind,target=/work/web/package-lock.json,src=./web/package-lock.json \
|
--mount=type=bind,target=/work/web/package-lock.json,src=./web/package-lock.json \
|
||||||
--mount=type=bind,target=/work/web/packages/sfe/package.json,src=./web/packages/sfe/package.json \
|
--mount=type=bind,target=/work/web/packages/sfe/package.json,src=./web/packages/sfe/package.json \
|
||||||
--mount=type=bind,target=/work/web/scripts,src=./web/scripts \
|
--mount=type=bind,target=/work/web/scripts,src=./web/scripts \
|
||||||
--mount=type=cache,id=npm-web,sharing=shared,target=/root/.npm \
|
--mount=type=cache,id=npm-ak,sharing=shared,target=/root/.npm \
|
||||||
npm ci --include=dev
|
npm ci --include=dev
|
||||||
|
|
||||||
COPY ./package.json /work
|
COPY ./package.json /work
|
||||||
@ -43,7 +24,7 @@ COPY ./gen-ts-api /work/web/node_modules/@goauthentik/api
|
|||||||
RUN npm run build && \
|
RUN npm run build && \
|
||||||
npm run build:sfe
|
npm run build:sfe
|
||||||
|
|
||||||
# Stage 3: Build go proxy
|
# Stage 2: Build go proxy
|
||||||
FROM --platform=${BUILDPLATFORM} docker.io/library/golang:1.24-bookworm AS go-builder
|
FROM --platform=${BUILDPLATFORM} docker.io/library/golang:1.24-bookworm AS go-builder
|
||||||
|
|
||||||
ARG TARGETOS
|
ARG TARGETOS
|
||||||
@ -68,8 +49,8 @@ RUN --mount=type=bind,target=/go/src/goauthentik.io/go.mod,src=./go.mod \
|
|||||||
COPY ./cmd /go/src/goauthentik.io/cmd
|
COPY ./cmd /go/src/goauthentik.io/cmd
|
||||||
COPY ./authentik/lib /go/src/goauthentik.io/authentik/lib
|
COPY ./authentik/lib /go/src/goauthentik.io/authentik/lib
|
||||||
COPY ./web/static.go /go/src/goauthentik.io/web/static.go
|
COPY ./web/static.go /go/src/goauthentik.io/web/static.go
|
||||||
COPY --from=web-builder /work/web/robots.txt /go/src/goauthentik.io/web/robots.txt
|
COPY --from=node-builder /work/web/robots.txt /go/src/goauthentik.io/web/robots.txt
|
||||||
COPY --from=web-builder /work/web/security.txt /go/src/goauthentik.io/web/security.txt
|
COPY --from=node-builder /work/web/security.txt /go/src/goauthentik.io/web/security.txt
|
||||||
COPY ./internal /go/src/goauthentik.io/internal
|
COPY ./internal /go/src/goauthentik.io/internal
|
||||||
COPY ./go.mod /go/src/goauthentik.io/go.mod
|
COPY ./go.mod /go/src/goauthentik.io/go.mod
|
||||||
COPY ./go.sum /go/src/goauthentik.io/go.sum
|
COPY ./go.sum /go/src/goauthentik.io/go.sum
|
||||||
@ -80,7 +61,7 @@ RUN --mount=type=cache,sharing=locked,target=/go/pkg/mod \
|
|||||||
CGO_ENABLED=1 GOFIPS140=latest GOARM="${TARGETVARIANT#v}" \
|
CGO_ENABLED=1 GOFIPS140=latest GOARM="${TARGETVARIANT#v}" \
|
||||||
go build -o /go/authentik ./cmd/server
|
go build -o /go/authentik ./cmd/server
|
||||||
|
|
||||||
# Stage 4: MaxMind GeoIP
|
# Stage 3: MaxMind GeoIP
|
||||||
FROM --platform=${BUILDPLATFORM} ghcr.io/maxmind/geoipupdate:v7.1.0 AS geoip
|
FROM --platform=${BUILDPLATFORM} ghcr.io/maxmind/geoipupdate:v7.1.0 AS geoip
|
||||||
|
|
||||||
ENV GEOIPUPDATE_EDITION_IDS="GeoLite2-City GeoLite2-ASN"
|
ENV GEOIPUPDATE_EDITION_IDS="GeoLite2-City GeoLite2-ASN"
|
||||||
@ -93,9 +74,9 @@ RUN --mount=type=secret,id=GEOIPUPDATE_ACCOUNT_ID \
|
|||||||
mkdir -p /usr/share/GeoIP && \
|
mkdir -p /usr/share/GeoIP && \
|
||||||
/bin/sh -c "GEOIPUPDATE_LICENSE_KEY_FILE=/run/secrets/GEOIPUPDATE_LICENSE_KEY /usr/bin/entry.sh || echo 'Failed to get GeoIP database, disabling'; exit 0"
|
/bin/sh -c "GEOIPUPDATE_LICENSE_KEY_FILE=/run/secrets/GEOIPUPDATE_LICENSE_KEY /usr/bin/entry.sh || echo 'Failed to get GeoIP database, disabling'; exit 0"
|
||||||
|
|
||||||
# Stage 5: Download uv
|
# Stage 4: Download uv
|
||||||
FROM ghcr.io/astral-sh/uv:0.7.8 AS uv
|
FROM ghcr.io/astral-sh/uv:0.7.11 AS uv
|
||||||
# Stage 6: Base python image
|
# Stage 5: Base python image
|
||||||
FROM ghcr.io/goauthentik/fips-python:3.13.3-slim-bookworm-fips AS python-base
|
FROM ghcr.io/goauthentik/fips-python:3.13.3-slim-bookworm-fips AS python-base
|
||||||
|
|
||||||
ENV VENV_PATH="/ak-root/.venv" \
|
ENV VENV_PATH="/ak-root/.venv" \
|
||||||
@ -109,7 +90,7 @@ WORKDIR /ak-root/
|
|||||||
|
|
||||||
COPY --from=uv /uv /uvx /bin/
|
COPY --from=uv /uv /uvx /bin/
|
||||||
|
|
||||||
# Stage 7: Python dependencies
|
# Stage 6: Python dependencies
|
||||||
FROM python-base AS python-deps
|
FROM python-base AS python-deps
|
||||||
|
|
||||||
ARG TARGETARCH
|
ARG TARGETARCH
|
||||||
@ -144,7 +125,7 @@ RUN --mount=type=bind,target=pyproject.toml,src=pyproject.toml \
|
|||||||
--mount=type=cache,target=/root/.cache/uv \
|
--mount=type=cache,target=/root/.cache/uv \
|
||||||
uv sync --frozen --no-install-project --no-dev
|
uv sync --frozen --no-install-project --no-dev
|
||||||
|
|
||||||
# Stage 8: Run
|
# Stage 7: Run
|
||||||
FROM python-base AS final-image
|
FROM python-base AS final-image
|
||||||
|
|
||||||
ARG VERSION
|
ARG VERSION
|
||||||
@ -187,9 +168,8 @@ COPY ./lifecycle/ /lifecycle
|
|||||||
COPY ./authentik/sources/kerberos/krb5.conf /etc/krb5.conf
|
COPY ./authentik/sources/kerberos/krb5.conf /etc/krb5.conf
|
||||||
COPY --from=go-builder /go/authentik /bin/authentik
|
COPY --from=go-builder /go/authentik /bin/authentik
|
||||||
COPY --from=python-deps /ak-root/.venv /ak-root/.venv
|
COPY --from=python-deps /ak-root/.venv /ak-root/.venv
|
||||||
COPY --from=web-builder /work/web/dist/ /web/dist/
|
COPY --from=node-builder /work/web/dist/ /web/dist/
|
||||||
COPY --from=web-builder /work/web/authentik/ /web/authentik/
|
COPY --from=node-builder /work/web/authentik/ /web/authentik/
|
||||||
COPY --from=website-builder /work/website/build/ /website/help/
|
|
||||||
COPY --from=geoip /usr/share/GeoIP /geoip
|
COPY --from=geoip /usr/share/GeoIP /geoip
|
||||||
|
|
||||||
USER 1000
|
USER 1000
|
||||||
|
@ -20,8 +20,8 @@ Even if the issue is not a CVE, we still greatly appreciate your help in hardeni
|
|||||||
|
|
||||||
| Version | Supported |
|
| Version | Supported |
|
||||||
| --------- | --------- |
|
| --------- | --------- |
|
||||||
| 2025.2.x | ✅ |
|
|
||||||
| 2025.4.x | ✅ |
|
| 2025.4.x | ✅ |
|
||||||
|
| 2025.6.x | ✅ |
|
||||||
|
|
||||||
## Reporting a Vulnerability
|
## Reporting a Vulnerability
|
||||||
|
|
||||||
|
@ -2,7 +2,7 @@
|
|||||||
|
|
||||||
from os import environ
|
from os import environ
|
||||||
|
|
||||||
__version__ = "2025.4.1"
|
__version__ = "2025.6.0"
|
||||||
ENV_GIT_HASH_KEY = "GIT_BUILD_HASH"
|
ENV_GIT_HASH_KEY = "GIT_BUILD_HASH"
|
||||||
|
|
||||||
|
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
"""authentik administration overview"""
|
"""authentik administration overview"""
|
||||||
|
|
||||||
from django.core.cache import cache
|
from django.core.cache import cache
|
||||||
|
from django_tenants.utils import get_public_schema_name
|
||||||
from drf_spectacular.utils import extend_schema
|
from drf_spectacular.utils import extend_schema
|
||||||
from packaging.version import parse
|
from packaging.version import parse
|
||||||
from rest_framework.fields import SerializerMethodField
|
from rest_framework.fields import SerializerMethodField
|
||||||
@ -13,6 +14,7 @@ from authentik import __version__, get_build_hash
|
|||||||
from authentik.admin.tasks import VERSION_CACHE_KEY, VERSION_NULL, update_latest_version
|
from authentik.admin.tasks import VERSION_CACHE_KEY, VERSION_NULL, update_latest_version
|
||||||
from authentik.core.api.utils import PassiveSerializer
|
from authentik.core.api.utils import PassiveSerializer
|
||||||
from authentik.outposts.models import Outpost
|
from authentik.outposts.models import Outpost
|
||||||
|
from authentik.tenants.utils import get_current_tenant
|
||||||
|
|
||||||
|
|
||||||
class VersionSerializer(PassiveSerializer):
|
class VersionSerializer(PassiveSerializer):
|
||||||
@ -35,6 +37,8 @@ class VersionSerializer(PassiveSerializer):
|
|||||||
|
|
||||||
def get_version_latest(self, _) -> str:
|
def get_version_latest(self, _) -> str:
|
||||||
"""Get latest version from cache"""
|
"""Get latest version from cache"""
|
||||||
|
if get_current_tenant().schema_name == get_public_schema_name():
|
||||||
|
return __version__
|
||||||
version_in_cache = cache.get(VERSION_CACHE_KEY)
|
version_in_cache = cache.get(VERSION_CACHE_KEY)
|
||||||
if not version_in_cache: # pragma: no cover
|
if not version_in_cache: # pragma: no cover
|
||||||
update_latest_version.delay()
|
update_latest_version.delay()
|
||||||
|
@ -14,3 +14,19 @@ class AuthentikAdminConfig(ManagedAppConfig):
|
|||||||
label = "authentik_admin"
|
label = "authentik_admin"
|
||||||
verbose_name = "authentik Admin"
|
verbose_name = "authentik Admin"
|
||||||
default = True
|
default = True
|
||||||
|
|
||||||
|
@ManagedAppConfig.reconcile_global
|
||||||
|
def clear_update_notifications(self):
|
||||||
|
"""Clear update notifications on startup if the notification was for the version
|
||||||
|
we're running now."""
|
||||||
|
from packaging.version import parse
|
||||||
|
|
||||||
|
from authentik.admin.tasks import LOCAL_VERSION
|
||||||
|
from authentik.events.models import EventAction, Notification
|
||||||
|
|
||||||
|
for notification in Notification.objects.filter(event__action=EventAction.UPDATE_AVAILABLE):
|
||||||
|
if "new_version" not in notification.event.context:
|
||||||
|
continue
|
||||||
|
notification_version = notification.event.context["new_version"]
|
||||||
|
if LOCAL_VERSION >= parse(notification_version):
|
||||||
|
notification.delete()
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
"""authentik admin settings"""
|
"""authentik admin settings"""
|
||||||
|
|
||||||
from celery.schedules import crontab
|
from celery.schedules import crontab
|
||||||
|
from django_tenants.utils import get_public_schema_name
|
||||||
|
|
||||||
from authentik.lib.utils.time import fqdn_rand
|
from authentik.lib.utils.time import fqdn_rand
|
||||||
|
|
||||||
@ -8,6 +9,7 @@ CELERY_BEAT_SCHEDULE = {
|
|||||||
"admin_latest_version": {
|
"admin_latest_version": {
|
||||||
"task": "authentik.admin.tasks.update_latest_version",
|
"task": "authentik.admin.tasks.update_latest_version",
|
||||||
"schedule": crontab(minute=fqdn_rand("admin_latest_version"), hour="*"),
|
"schedule": crontab(minute=fqdn_rand("admin_latest_version"), hour="*"),
|
||||||
|
"tenant_schemas": [get_public_schema_name()],
|
||||||
"options": {"queue": "authentik_scheduled"},
|
"options": {"queue": "authentik_scheduled"},
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -1,7 +1,6 @@
|
|||||||
"""authentik admin tasks"""
|
"""authentik admin tasks"""
|
||||||
|
|
||||||
from django.core.cache import cache
|
from django.core.cache import cache
|
||||||
from django.db import DatabaseError, InternalError, ProgrammingError
|
|
||||||
from django.utils.translation import gettext_lazy as _
|
from django.utils.translation import gettext_lazy as _
|
||||||
from packaging.version import parse
|
from packaging.version import parse
|
||||||
from requests import RequestException
|
from requests import RequestException
|
||||||
@ -9,7 +8,7 @@ from structlog.stdlib import get_logger
|
|||||||
|
|
||||||
from authentik import __version__, get_build_hash
|
from authentik import __version__, get_build_hash
|
||||||
from authentik.admin.apps import PROM_INFO
|
from authentik.admin.apps import PROM_INFO
|
||||||
from authentik.events.models import Event, EventAction, Notification
|
from authentik.events.models import Event, EventAction
|
||||||
from authentik.events.system_tasks import SystemTask, TaskStatus, prefill_task
|
from authentik.events.system_tasks import SystemTask, TaskStatus, prefill_task
|
||||||
from authentik.lib.config import CONFIG
|
from authentik.lib.config import CONFIG
|
||||||
from authentik.lib.utils.http import get_http_session
|
from authentik.lib.utils.http import get_http_session
|
||||||
@ -33,20 +32,6 @@ def _set_prom_info():
|
|||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
@CELERY_APP.task(
|
|
||||||
throws=(DatabaseError, ProgrammingError, InternalError),
|
|
||||||
)
|
|
||||||
def clear_update_notifications():
|
|
||||||
"""Clear update notifications on startup if the notification was for the version
|
|
||||||
we're running now."""
|
|
||||||
for notification in Notification.objects.filter(event__action=EventAction.UPDATE_AVAILABLE):
|
|
||||||
if "new_version" not in notification.event.context:
|
|
||||||
continue
|
|
||||||
notification_version = notification.event.context["new_version"]
|
|
||||||
if LOCAL_VERSION >= parse(notification_version):
|
|
||||||
notification.delete()
|
|
||||||
|
|
||||||
|
|
||||||
@CELERY_APP.task(bind=True, base=SystemTask)
|
@CELERY_APP.task(bind=True, base=SystemTask)
|
||||||
@prefill_task
|
@prefill_task
|
||||||
def update_latest_version(self: SystemTask):
|
def update_latest_version(self: SystemTask):
|
||||||
|
@ -1,12 +1,12 @@
|
|||||||
"""test admin tasks"""
|
"""test admin tasks"""
|
||||||
|
|
||||||
|
from django.apps import apps
|
||||||
from django.core.cache import cache
|
from django.core.cache import cache
|
||||||
from django.test import TestCase
|
from django.test import TestCase
|
||||||
from requests_mock import Mocker
|
from requests_mock import Mocker
|
||||||
|
|
||||||
from authentik.admin.tasks import (
|
from authentik.admin.tasks import (
|
||||||
VERSION_CACHE_KEY,
|
VERSION_CACHE_KEY,
|
||||||
clear_update_notifications,
|
|
||||||
update_latest_version,
|
update_latest_version,
|
||||||
)
|
)
|
||||||
from authentik.events.models import Event, EventAction
|
from authentik.events.models import Event, EventAction
|
||||||
@ -72,12 +72,13 @@ class TestAdminTasks(TestCase):
|
|||||||
|
|
||||||
def test_clear_update_notifications(self):
|
def test_clear_update_notifications(self):
|
||||||
"""Test clear of previous notification"""
|
"""Test clear of previous notification"""
|
||||||
|
admin_config = apps.get_app_config("authentik_admin")
|
||||||
Event.objects.create(
|
Event.objects.create(
|
||||||
action=EventAction.UPDATE_AVAILABLE, context={"new_version": "99999999.9999999.9999999"}
|
action=EventAction.UPDATE_AVAILABLE, context={"new_version": "99999999.9999999.9999999"}
|
||||||
)
|
)
|
||||||
Event.objects.create(action=EventAction.UPDATE_AVAILABLE, context={"new_version": "1.1.1"})
|
Event.objects.create(action=EventAction.UPDATE_AVAILABLE, context={"new_version": "1.1.1"})
|
||||||
Event.objects.create(action=EventAction.UPDATE_AVAILABLE, context={})
|
Event.objects.create(action=EventAction.UPDATE_AVAILABLE, context={})
|
||||||
clear_update_notifications()
|
admin_config.clear_update_notifications()
|
||||||
self.assertFalse(
|
self.assertFalse(
|
||||||
Event.objects.filter(
|
Event.objects.filter(
|
||||||
action=EventAction.UPDATE_AVAILABLE, context__new_version="1.1"
|
action=EventAction.UPDATE_AVAILABLE, context__new_version="1.1"
|
||||||
|
@ -1,12 +1,13 @@
|
|||||||
"""authentik API AppConfig"""
|
"""authentik API AppConfig"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikAPIConfig(AppConfig):
|
class AuthentikAPIConfig(ManagedAppConfig):
|
||||||
"""authentik API Config"""
|
"""authentik API Config"""
|
||||||
|
|
||||||
name = "authentik.api"
|
name = "authentik.api"
|
||||||
label = "authentik_api"
|
label = "authentik_api"
|
||||||
mountpoint = "api/"
|
mountpoint = "api/"
|
||||||
verbose_name = "authentik API"
|
verbose_name = "authentik API"
|
||||||
|
default = True
|
||||||
|
14
authentik/blueprints/tests/test_managed_app_config.py
Normal file
14
authentik/blueprints/tests/test_managed_app_config.py
Normal file
@ -0,0 +1,14 @@
|
|||||||
|
from django.test import TestCase
|
||||||
|
|
||||||
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
from authentik.enterprise.apps import EnterpriseConfig
|
||||||
|
from authentik.lib.utils.reflection import get_apps
|
||||||
|
|
||||||
|
|
||||||
|
class TestManagedAppConfig(TestCase):
|
||||||
|
def test_apps_use_managed_app_config(self):
|
||||||
|
for app in get_apps():
|
||||||
|
if app.name.startswith("authentik.enterprise"):
|
||||||
|
self.assertIn(EnterpriseConfig, app.__class__.__bases__)
|
||||||
|
else:
|
||||||
|
self.assertIn(ManagedAppConfig, app.__class__.__bases__)
|
@ -1,9 +1,9 @@
|
|||||||
"""authentik brands app"""
|
"""authentik brands app"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikBrandsConfig(AppConfig):
|
class AuthentikBrandsConfig(ManagedAppConfig):
|
||||||
"""authentik Brand app"""
|
"""authentik Brand app"""
|
||||||
|
|
||||||
name = "authentik.brands"
|
name = "authentik.brands"
|
||||||
@ -12,3 +12,4 @@ class AuthentikBrandsConfig(AppConfig):
|
|||||||
mountpoints = {
|
mountpoints = {
|
||||||
"authentik.brands.urls_root": "",
|
"authentik.brands.urls_root": "",
|
||||||
}
|
}
|
||||||
|
default = True
|
||||||
|
@ -153,10 +153,10 @@ class ApplicationViewSet(UsedByMixin, ModelViewSet):
|
|||||||
return applications
|
return applications
|
||||||
|
|
||||||
def _filter_applications_with_launch_url(
|
def _filter_applications_with_launch_url(
|
||||||
self, pagined_apps: Iterator[Application]
|
self, paginated_apps: Iterator[Application]
|
||||||
) -> list[Application]:
|
) -> list[Application]:
|
||||||
applications = []
|
applications = []
|
||||||
for app in pagined_apps:
|
for app in paginated_apps:
|
||||||
if app.get_launch_url():
|
if app.get_launch_url():
|
||||||
applications.append(app)
|
applications.append(app)
|
||||||
return applications
|
return applications
|
||||||
|
@ -84,6 +84,7 @@ from authentik.flows.views.executor import QS_KEY_TOKEN
|
|||||||
from authentik.lib.avatars import get_avatar
|
from authentik.lib.avatars import get_avatar
|
||||||
from authentik.rbac.decorators import permission_required
|
from authentik.rbac.decorators import permission_required
|
||||||
from authentik.rbac.models import get_permission_choices
|
from authentik.rbac.models import get_permission_choices
|
||||||
|
from authentik.stages.email.flow import pickle_flow_token_for_email
|
||||||
from authentik.stages.email.models import EmailStage
|
from authentik.stages.email.models import EmailStage
|
||||||
from authentik.stages.email.tasks import send_mails
|
from authentik.stages.email.tasks import send_mails
|
||||||
from authentik.stages.email.utils import TemplateEmailMessage
|
from authentik.stages.email.utils import TemplateEmailMessage
|
||||||
@ -451,7 +452,7 @@ class UserViewSet(UsedByMixin, ModelViewSet):
|
|||||||
def list(self, request, *args, **kwargs):
|
def list(self, request, *args, **kwargs):
|
||||||
return super().list(request, *args, **kwargs)
|
return super().list(request, *args, **kwargs)
|
||||||
|
|
||||||
def _create_recovery_link(self) -> tuple[str, Token]:
|
def _create_recovery_link(self, for_email=False) -> tuple[str, Token]:
|
||||||
"""Create a recovery link (when the current brand has a recovery flow set),
|
"""Create a recovery link (when the current brand has a recovery flow set),
|
||||||
that can either be shown to an admin or sent to the user directly"""
|
that can either be shown to an admin or sent to the user directly"""
|
||||||
brand: Brand = self.request._request.brand
|
brand: Brand = self.request._request.brand
|
||||||
@ -473,12 +474,16 @@ class UserViewSet(UsedByMixin, ModelViewSet):
|
|||||||
raise ValidationError(
|
raise ValidationError(
|
||||||
{"non_field_errors": "Recovery flow not applicable to user"}
|
{"non_field_errors": "Recovery flow not applicable to user"}
|
||||||
) from None
|
) from None
|
||||||
|
_plan = FlowToken.pickle(plan)
|
||||||
|
if for_email:
|
||||||
|
_plan = pickle_flow_token_for_email(plan)
|
||||||
token, __ = FlowToken.objects.update_or_create(
|
token, __ = FlowToken.objects.update_or_create(
|
||||||
identifier=f"{user.uid}-password-reset",
|
identifier=f"{user.uid}-password-reset",
|
||||||
defaults={
|
defaults={
|
||||||
"user": user,
|
"user": user,
|
||||||
"flow": flow,
|
"flow": flow,
|
||||||
"_plan": FlowToken.pickle(plan),
|
"_plan": _plan,
|
||||||
|
"revoke_on_execution": not for_email,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
querystring = urlencode({QS_KEY_TOKEN: token.key})
|
querystring = urlencode({QS_KEY_TOKEN: token.key})
|
||||||
@ -648,7 +653,7 @@ class UserViewSet(UsedByMixin, ModelViewSet):
|
|||||||
if for_user.email == "":
|
if for_user.email == "":
|
||||||
LOGGER.debug("User doesn't have an email address")
|
LOGGER.debug("User doesn't have an email address")
|
||||||
raise ValidationError({"non_field_errors": "User does not have an email address set."})
|
raise ValidationError({"non_field_errors": "User does not have an email address set."})
|
||||||
link, token = self._create_recovery_link()
|
link, token = self._create_recovery_link(for_email=True)
|
||||||
# Lookup the email stage to assure the current user can access it
|
# Lookup the email stage to assure the current user can access it
|
||||||
stages = get_objects_for_user(
|
stages = get_objects_for_user(
|
||||||
request.user, "authentik_stages_email.view_emailstage"
|
request.user, "authentik_stages_email.view_emailstage"
|
||||||
|
@ -79,6 +79,7 @@ def _migrate_session(
|
|||||||
AuthenticatedSession.objects.using(db_alias).create(
|
AuthenticatedSession.objects.using(db_alias).create(
|
||||||
session=session,
|
session=session,
|
||||||
user=old_auth_session.user,
|
user=old_auth_session.user,
|
||||||
|
uuid=old_auth_session.uuid,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
@ -1,10 +1,81 @@
|
|||||||
# Generated by Django 5.1.9 on 2025-05-14 11:15
|
# Generated by Django 5.1.9 on 2025-05-14 11:15
|
||||||
|
|
||||||
from django.apps.registry import Apps
|
from django.apps.registry import Apps, apps as global_apps
|
||||||
from django.db import migrations
|
from django.db import migrations
|
||||||
|
from django.contrib.contenttypes.management import create_contenttypes
|
||||||
|
from django.contrib.auth.management import create_permissions
|
||||||
from django.db.backends.base.schema import BaseDatabaseSchemaEditor
|
from django.db.backends.base.schema import BaseDatabaseSchemaEditor
|
||||||
|
|
||||||
|
|
||||||
|
def migrate_authenticated_session_permissions(apps: Apps, schema_editor: BaseDatabaseSchemaEditor):
|
||||||
|
"""Migrate permissions from OldAuthenticatedSession to AuthenticatedSession"""
|
||||||
|
db_alias = schema_editor.connection.alias
|
||||||
|
|
||||||
|
# `apps` here is just an instance of `django.db.migrations.state.AppConfigStub`, we need the
|
||||||
|
# real config for creating permissions and content types
|
||||||
|
authentik_core_config = global_apps.get_app_config("authentik_core")
|
||||||
|
# These are only ran by django after all migrations, but we need them right now.
|
||||||
|
# `global_apps` is needed,
|
||||||
|
create_permissions(authentik_core_config, using=db_alias, verbosity=1)
|
||||||
|
create_contenttypes(authentik_core_config, using=db_alias, verbosity=1)
|
||||||
|
|
||||||
|
# But from now on, this is just a regular migration, so use `apps`
|
||||||
|
Permission = apps.get_model("auth", "Permission")
|
||||||
|
ContentType = apps.get_model("contenttypes", "ContentType")
|
||||||
|
|
||||||
|
try:
|
||||||
|
old_ct = ContentType.objects.using(db_alias).get(
|
||||||
|
app_label="authentik_core", model="oldauthenticatedsession"
|
||||||
|
)
|
||||||
|
new_ct = ContentType.objects.using(db_alias).get(
|
||||||
|
app_label="authentik_core", model="authenticatedsession"
|
||||||
|
)
|
||||||
|
except ContentType.DoesNotExist:
|
||||||
|
# This should exist at this point, but if not, let's cut our losses
|
||||||
|
return
|
||||||
|
|
||||||
|
# Get all permissions for the old content type
|
||||||
|
old_perms = Permission.objects.using(db_alias).filter(content_type=old_ct)
|
||||||
|
|
||||||
|
# Create equivalent permissions for the new content type
|
||||||
|
for old_perm in old_perms:
|
||||||
|
new_perm = (
|
||||||
|
Permission.objects.using(db_alias)
|
||||||
|
.filter(
|
||||||
|
content_type=new_ct,
|
||||||
|
codename=old_perm.codename,
|
||||||
|
)
|
||||||
|
.first()
|
||||||
|
)
|
||||||
|
if not new_perm:
|
||||||
|
# This should exist at this point, but if not, let's cut our losses
|
||||||
|
continue
|
||||||
|
|
||||||
|
# Global user permissions
|
||||||
|
User = apps.get_model("authentik_core", "User")
|
||||||
|
User.user_permissions.through.objects.using(db_alias).filter(
|
||||||
|
permission=old_perm
|
||||||
|
).all().update(permission=new_perm)
|
||||||
|
|
||||||
|
# Global role permissions
|
||||||
|
DjangoGroup = apps.get_model("auth", "Group")
|
||||||
|
DjangoGroup.permissions.through.objects.using(db_alias).filter(
|
||||||
|
permission=old_perm
|
||||||
|
).all().update(permission=new_perm)
|
||||||
|
|
||||||
|
# Object user permissions
|
||||||
|
UserObjectPermission = apps.get_model("guardian", "UserObjectPermission")
|
||||||
|
UserObjectPermission.objects.using(db_alias).filter(permission=old_perm).all().update(
|
||||||
|
permission=new_perm, content_type=new_ct
|
||||||
|
)
|
||||||
|
|
||||||
|
# Object role permissions
|
||||||
|
GroupObjectPermission = apps.get_model("guardian", "GroupObjectPermission")
|
||||||
|
GroupObjectPermission.objects.using(db_alias).filter(permission=old_perm).all().update(
|
||||||
|
permission=new_perm, content_type=new_ct
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
def remove_old_authenticated_session_content_type(
|
def remove_old_authenticated_session_content_type(
|
||||||
apps: Apps, schema_editor: BaseDatabaseSchemaEditor
|
apps: Apps, schema_editor: BaseDatabaseSchemaEditor
|
||||||
):
|
):
|
||||||
@ -21,7 +92,12 @@ class Migration(migrations.Migration):
|
|||||||
]
|
]
|
||||||
|
|
||||||
operations = [
|
operations = [
|
||||||
|
migrations.RunPython(
|
||||||
|
code=migrate_authenticated_session_permissions,
|
||||||
|
reverse_code=migrations.RunPython.noop,
|
||||||
|
),
|
||||||
migrations.RunPython(
|
migrations.RunPython(
|
||||||
code=remove_old_authenticated_session_content_type,
|
code=remove_old_authenticated_session_content_type,
|
||||||
|
reverse_code=migrations.RunPython.noop,
|
||||||
),
|
),
|
||||||
]
|
]
|
||||||
|
@ -10,7 +10,7 @@
|
|||||||
{% endblock %}
|
{% endblock %}
|
||||||
|
|
||||||
{% block body %}
|
{% block body %}
|
||||||
<ak-message-container></ak-message-container>
|
<ak-message-container alignment="bottom"></ak-message-container>
|
||||||
<ak-interface-admin>
|
<ak-interface-admin>
|
||||||
<ak-loading></ak-loading>
|
<ak-loading></ak-loading>
|
||||||
</ak-interface-admin>
|
</ak-interface-admin>
|
||||||
|
@ -0,0 +1,18 @@
|
|||||||
|
# Generated by Django 5.1.9 on 2025-05-27 12:52
|
||||||
|
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
|
dependencies = [
|
||||||
|
("authentik_flows", "0027_auto_20231028_1424"),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AddField(
|
||||||
|
model_name="flowtoken",
|
||||||
|
name="revoke_on_execution",
|
||||||
|
field=models.BooleanField(default=True),
|
||||||
|
),
|
||||||
|
]
|
@ -303,9 +303,10 @@ class FlowToken(Token):
|
|||||||
|
|
||||||
flow = models.ForeignKey(Flow, on_delete=models.CASCADE)
|
flow = models.ForeignKey(Flow, on_delete=models.CASCADE)
|
||||||
_plan = models.TextField()
|
_plan = models.TextField()
|
||||||
|
revoke_on_execution = models.BooleanField(default=True)
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def pickle(plan) -> str:
|
def pickle(plan: "FlowPlan") -> str:
|
||||||
"""Pickle into string"""
|
"""Pickle into string"""
|
||||||
data = dumps(plan)
|
data = dumps(plan)
|
||||||
return b64encode(data).decode()
|
return b64encode(data).decode()
|
||||||
|
@ -99,9 +99,10 @@ class ChallengeStageView(StageView):
|
|||||||
self.logger.debug("Got StageInvalidException", exc=exc)
|
self.logger.debug("Got StageInvalidException", exc=exc)
|
||||||
return self.executor.stage_invalid()
|
return self.executor.stage_invalid()
|
||||||
if not challenge.is_valid():
|
if not challenge.is_valid():
|
||||||
self.logger.warning(
|
self.logger.error(
|
||||||
"f(ch): Invalid challenge",
|
"f(ch): Invalid challenge",
|
||||||
errors=challenge.errors,
|
errors=challenge.errors,
|
||||||
|
challenge=challenge.data,
|
||||||
)
|
)
|
||||||
return HttpChallengeResponse(challenge)
|
return HttpChallengeResponse(challenge)
|
||||||
|
|
||||||
|
@ -146,7 +146,8 @@ class FlowExecutorView(APIView):
|
|||||||
except (AttributeError, EOFError, ImportError, IndexError) as exc:
|
except (AttributeError, EOFError, ImportError, IndexError) as exc:
|
||||||
LOGGER.warning("f(exec): Failed to restore token plan", exc=exc)
|
LOGGER.warning("f(exec): Failed to restore token plan", exc=exc)
|
||||||
finally:
|
finally:
|
||||||
token.delete()
|
if token.revoke_on_execution:
|
||||||
|
token.delete()
|
||||||
if not isinstance(plan, FlowPlan):
|
if not isinstance(plan, FlowPlan):
|
||||||
return None
|
return None
|
||||||
plan.context[PLAN_CONTEXT_IS_RESTORED] = token
|
plan.context[PLAN_CONTEXT_IS_RESTORED] = token
|
||||||
|
@ -81,7 +81,6 @@ debugger: false
|
|||||||
|
|
||||||
log_level: info
|
log_level: info
|
||||||
|
|
||||||
session_storage: cache
|
|
||||||
sessions:
|
sessions:
|
||||||
unauthenticated_age: days=1
|
unauthenticated_age: days=1
|
||||||
|
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
from collections.abc import Callable
|
from collections.abc import Callable
|
||||||
from dataclasses import asdict
|
from dataclasses import asdict
|
||||||
|
|
||||||
|
from celery import group
|
||||||
from celery.exceptions import Retry
|
from celery.exceptions import Retry
|
||||||
from celery.result import allow_join_result
|
from celery.result import allow_join_result
|
||||||
from django.core.paginator import Paginator
|
from django.core.paginator import Paginator
|
||||||
@ -82,21 +83,41 @@ class SyncTasks:
|
|||||||
self.logger.debug("Failed to acquire sync lock, skipping", provider=provider.name)
|
self.logger.debug("Failed to acquire sync lock, skipping", provider=provider.name)
|
||||||
return
|
return
|
||||||
try:
|
try:
|
||||||
for page in users_paginator.page_range:
|
messages.append(_("Syncing users"))
|
||||||
messages.append(_("Syncing page {page} of users".format(page=page)))
|
user_results = (
|
||||||
for msg in sync_objects.apply_async(
|
group(
|
||||||
args=(class_to_path(User), page, provider_pk),
|
[
|
||||||
time_limit=PAGE_TIMEOUT,
|
sync_objects.signature(
|
||||||
soft_time_limit=PAGE_TIMEOUT,
|
args=(class_to_path(User), page, provider_pk),
|
||||||
).get():
|
time_limit=PAGE_TIMEOUT,
|
||||||
|
soft_time_limit=PAGE_TIMEOUT,
|
||||||
|
)
|
||||||
|
for page in users_paginator.page_range
|
||||||
|
]
|
||||||
|
)
|
||||||
|
.apply_async()
|
||||||
|
.get()
|
||||||
|
)
|
||||||
|
for result in user_results:
|
||||||
|
for msg in result:
|
||||||
messages.append(LogEvent(**msg))
|
messages.append(LogEvent(**msg))
|
||||||
for page in groups_paginator.page_range:
|
messages.append(_("Syncing groups"))
|
||||||
messages.append(_("Syncing page {page} of groups".format(page=page)))
|
group_results = (
|
||||||
for msg in sync_objects.apply_async(
|
group(
|
||||||
args=(class_to_path(Group), page, provider_pk),
|
[
|
||||||
time_limit=PAGE_TIMEOUT,
|
sync_objects.signature(
|
||||||
soft_time_limit=PAGE_TIMEOUT,
|
args=(class_to_path(Group), page, provider_pk),
|
||||||
).get():
|
time_limit=PAGE_TIMEOUT,
|
||||||
|
soft_time_limit=PAGE_TIMEOUT,
|
||||||
|
)
|
||||||
|
for page in groups_paginator.page_range
|
||||||
|
]
|
||||||
|
)
|
||||||
|
.apply_async()
|
||||||
|
.get()
|
||||||
|
)
|
||||||
|
for result in group_results:
|
||||||
|
for msg in result:
|
||||||
messages.append(LogEvent(**msg))
|
messages.append(LogEvent(**msg))
|
||||||
except TransientSyncException as exc:
|
except TransientSyncException as exc:
|
||||||
self.logger.warning("transient sync exception", exc=exc)
|
self.logger.warning("transient sync exception", exc=exc)
|
||||||
@ -109,7 +130,7 @@ class SyncTasks:
|
|||||||
def sync_objects(
|
def sync_objects(
|
||||||
self, object_type: str, page: int, provider_pk: int, override_dry_run=False, **filter
|
self, object_type: str, page: int, provider_pk: int, override_dry_run=False, **filter
|
||||||
):
|
):
|
||||||
_object_type = path_to_class(object_type)
|
_object_type: type[Model] = path_to_class(object_type)
|
||||||
self.logger = get_logger().bind(
|
self.logger = get_logger().bind(
|
||||||
provider_type=class_to_path(self._provider_model),
|
provider_type=class_to_path(self._provider_model),
|
||||||
provider_pk=provider_pk,
|
provider_pk=provider_pk,
|
||||||
@ -132,6 +153,19 @@ class SyncTasks:
|
|||||||
self.logger.debug("starting discover")
|
self.logger.debug("starting discover")
|
||||||
client.discover()
|
client.discover()
|
||||||
self.logger.debug("starting sync for page", page=page)
|
self.logger.debug("starting sync for page", page=page)
|
||||||
|
messages.append(
|
||||||
|
asdict(
|
||||||
|
LogEvent(
|
||||||
|
_(
|
||||||
|
"Syncing page {page} of {object_type}".format(
|
||||||
|
page=page, object_type=_object_type._meta.verbose_name_plural
|
||||||
|
)
|
||||||
|
),
|
||||||
|
log_level="info",
|
||||||
|
logger=f"{provider._meta.verbose_name}@{object_type}",
|
||||||
|
)
|
||||||
|
)
|
||||||
|
)
|
||||||
for obj in paginator.page(page).object_list:
|
for obj in paginator.page(page).object_list:
|
||||||
obj: Model
|
obj: Model
|
||||||
try:
|
try:
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""Authentik policy dummy app config"""
|
"""Authentik policy dummy app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikPolicyDummyConfig(AppConfig):
|
class AuthentikPolicyDummyConfig(ManagedAppConfig):
|
||||||
"""Authentik policy_dummy app config"""
|
"""Authentik policy_dummy app config"""
|
||||||
|
|
||||||
name = "authentik.policies.dummy"
|
name = "authentik.policies.dummy"
|
||||||
label = "authentik_policies_dummy"
|
label = "authentik_policies_dummy"
|
||||||
verbose_name = "authentik Policies.Dummy"
|
verbose_name = "authentik Policies.Dummy"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,11 @@
|
|||||||
"""authentik policy engine"""
|
"""authentik policy engine"""
|
||||||
|
|
||||||
from collections.abc import Iterator
|
from collections.abc import Iterable
|
||||||
from multiprocessing import Pipe, current_process
|
from multiprocessing import Pipe, current_process
|
||||||
from multiprocessing.connection import Connection
|
from multiprocessing.connection import Connection
|
||||||
from time import perf_counter
|
|
||||||
|
|
||||||
from django.core.cache import cache
|
from django.core.cache import cache
|
||||||
|
from django.db.models import Count, Q, QuerySet
|
||||||
from django.http import HttpRequest
|
from django.http import HttpRequest
|
||||||
from sentry_sdk import start_span
|
from sentry_sdk import start_span
|
||||||
from sentry_sdk.tracing import Span
|
from sentry_sdk.tracing import Span
|
||||||
@ -67,14 +67,11 @@ class PolicyEngine:
|
|||||||
self.__processes: list[PolicyProcessInfo] = []
|
self.__processes: list[PolicyProcessInfo] = []
|
||||||
self.use_cache = True
|
self.use_cache = True
|
||||||
self.__expected_result_count = 0
|
self.__expected_result_count = 0
|
||||||
|
self.__static_result: PolicyResult | None = None
|
||||||
|
|
||||||
def iterate_bindings(self) -> Iterator[PolicyBinding]:
|
def bindings(self) -> QuerySet[PolicyBinding] | Iterable[PolicyBinding]:
|
||||||
"""Make sure all Policies are their respective classes"""
|
"""Make sure all Policies are their respective classes"""
|
||||||
return (
|
return PolicyBinding.objects.filter(target=self.__pbm, enabled=True).order_by("order")
|
||||||
PolicyBinding.objects.filter(target=self.__pbm, enabled=True)
|
|
||||||
.order_by("order")
|
|
||||||
.iterator()
|
|
||||||
)
|
|
||||||
|
|
||||||
def _check_policy_type(self, binding: PolicyBinding):
|
def _check_policy_type(self, binding: PolicyBinding):
|
||||||
"""Check policy type, make sure it's not the root class as that has no logic implemented"""
|
"""Check policy type, make sure it's not the root class as that has no logic implemented"""
|
||||||
@ -84,30 +81,66 @@ class PolicyEngine:
|
|||||||
def _check_cache(self, binding: PolicyBinding):
|
def _check_cache(self, binding: PolicyBinding):
|
||||||
if not self.use_cache:
|
if not self.use_cache:
|
||||||
return False
|
return False
|
||||||
before = perf_counter()
|
# It's a bit silly to time this, but
|
||||||
key = cache_key(binding, self.request)
|
with HIST_POLICIES_EXECUTION_TIME.labels(
|
||||||
cached_policy = cache.get(key, None)
|
|
||||||
duration = max(perf_counter() - before, 0)
|
|
||||||
if not cached_policy:
|
|
||||||
return False
|
|
||||||
self.logger.debug(
|
|
||||||
"P_ENG: Taking result from cache",
|
|
||||||
binding=binding,
|
|
||||||
cache_key=key,
|
|
||||||
request=self.request,
|
|
||||||
)
|
|
||||||
HIST_POLICIES_EXECUTION_TIME.labels(
|
|
||||||
binding_order=binding.order,
|
binding_order=binding.order,
|
||||||
binding_target_type=binding.target_type,
|
binding_target_type=binding.target_type,
|
||||||
binding_target_name=binding.target_name,
|
binding_target_name=binding.target_name,
|
||||||
object_pk=str(self.request.obj.pk),
|
object_pk=str(self.request.obj.pk),
|
||||||
object_type=class_to_path(self.request.obj.__class__),
|
object_type=class_to_path(self.request.obj.__class__),
|
||||||
mode="cache_retrieve",
|
mode="cache_retrieve",
|
||||||
).observe(duration)
|
).time():
|
||||||
# It's a bit silly to time this, but
|
key = cache_key(binding, self.request)
|
||||||
|
cached_policy = cache.get(key, None)
|
||||||
|
if not cached_policy:
|
||||||
|
return False
|
||||||
|
self.logger.debug(
|
||||||
|
"P_ENG: Taking result from cache",
|
||||||
|
binding=binding,
|
||||||
|
cache_key=key,
|
||||||
|
request=self.request,
|
||||||
|
)
|
||||||
self.__cached_policies.append(cached_policy)
|
self.__cached_policies.append(cached_policy)
|
||||||
return True
|
return True
|
||||||
|
|
||||||
|
def compute_static_bindings(self, bindings: QuerySet[PolicyBinding]):
|
||||||
|
"""Check static bindings if possible"""
|
||||||
|
aggrs = {
|
||||||
|
"total": Count(
|
||||||
|
"pk", filter=Q(Q(group__isnull=False) | Q(user__isnull=False), policy=None)
|
||||||
|
),
|
||||||
|
}
|
||||||
|
if self.request.user.pk:
|
||||||
|
all_groups = self.request.user.all_groups()
|
||||||
|
aggrs["passing"] = Count(
|
||||||
|
"pk",
|
||||||
|
filter=Q(
|
||||||
|
Q(
|
||||||
|
Q(user=self.request.user) | Q(group__in=all_groups),
|
||||||
|
negate=False,
|
||||||
|
)
|
||||||
|
| Q(
|
||||||
|
Q(~Q(user=self.request.user), user__isnull=False)
|
||||||
|
| Q(~Q(group__in=all_groups), group__isnull=False),
|
||||||
|
negate=True,
|
||||||
|
),
|
||||||
|
enabled=True,
|
||||||
|
),
|
||||||
|
)
|
||||||
|
matched_bindings = bindings.aggregate(**aggrs)
|
||||||
|
passing = False
|
||||||
|
if matched_bindings["total"] == 0 and matched_bindings.get("passing", 0) == 0:
|
||||||
|
# If we didn't find any static bindings, do nothing
|
||||||
|
return
|
||||||
|
self.logger.debug("P_ENG: Found static bindings", **matched_bindings)
|
||||||
|
if matched_bindings.get("passing", 0) > 0:
|
||||||
|
# Any passing static binding -> passing
|
||||||
|
passing = True
|
||||||
|
elif matched_bindings["total"] > 0 and matched_bindings.get("passing", 0) < 1:
|
||||||
|
# No matching static bindings but at least one is configured -> not passing
|
||||||
|
passing = False
|
||||||
|
self.__static_result = PolicyResult(passing)
|
||||||
|
|
||||||
def build(self) -> "PolicyEngine":
|
def build(self) -> "PolicyEngine":
|
||||||
"""Build wrapper which monitors performance"""
|
"""Build wrapper which monitors performance"""
|
||||||
with (
|
with (
|
||||||
@ -123,7 +156,12 @@ class PolicyEngine:
|
|||||||
span: Span
|
span: Span
|
||||||
span.set_data("pbm", self.__pbm)
|
span.set_data("pbm", self.__pbm)
|
||||||
span.set_data("request", self.request)
|
span.set_data("request", self.request)
|
||||||
for binding in self.iterate_bindings():
|
bindings = self.bindings()
|
||||||
|
policy_bindings = bindings
|
||||||
|
if isinstance(bindings, QuerySet):
|
||||||
|
self.compute_static_bindings(bindings)
|
||||||
|
policy_bindings = [x for x in bindings if x.policy]
|
||||||
|
for binding in policy_bindings:
|
||||||
self.__expected_result_count += 1
|
self.__expected_result_count += 1
|
||||||
|
|
||||||
self._check_policy_type(binding)
|
self._check_policy_type(binding)
|
||||||
@ -153,10 +191,13 @@ class PolicyEngine:
|
|||||||
@property
|
@property
|
||||||
def result(self) -> PolicyResult:
|
def result(self) -> PolicyResult:
|
||||||
"""Get policy-checking result"""
|
"""Get policy-checking result"""
|
||||||
|
self.__processes.sort(key=lambda x: x.binding.order)
|
||||||
process_results: list[PolicyResult] = [x.result for x in self.__processes if x.result]
|
process_results: list[PolicyResult] = [x.result for x in self.__processes if x.result]
|
||||||
all_results = list(process_results + self.__cached_policies)
|
all_results = list(process_results + self.__cached_policies)
|
||||||
if len(all_results) < self.__expected_result_count: # pragma: no cover
|
if len(all_results) < self.__expected_result_count: # pragma: no cover
|
||||||
raise AssertionError("Got less results than polices")
|
raise AssertionError("Got less results than polices")
|
||||||
|
if self.__static_result:
|
||||||
|
all_results.append(self.__static_result)
|
||||||
# No results, no policies attached -> passing
|
# No results, no policies attached -> passing
|
||||||
if len(all_results) == 0:
|
if len(all_results) == 0:
|
||||||
return PolicyResult(self.empty_result)
|
return PolicyResult(self.empty_result)
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik Event Matcher policy app config"""
|
"""authentik Event Matcher policy app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikPoliciesEventMatcherConfig(AppConfig):
|
class AuthentikPoliciesEventMatcherConfig(ManagedAppConfig):
|
||||||
"""authentik Event Matcher policy app config"""
|
"""authentik Event Matcher policy app config"""
|
||||||
|
|
||||||
name = "authentik.policies.event_matcher"
|
name = "authentik.policies.event_matcher"
|
||||||
label = "authentik_policies_event_matcher"
|
label = "authentik_policies_event_matcher"
|
||||||
verbose_name = "authentik Policies.Event Matcher"
|
verbose_name = "authentik Policies.Event Matcher"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""Authentik policy_expiry app config"""
|
"""Authentik policy_expiry app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikPolicyExpiryConfig(AppConfig):
|
class AuthentikPolicyExpiryConfig(ManagedAppConfig):
|
||||||
"""Authentik policy_expiry app config"""
|
"""Authentik policy_expiry app config"""
|
||||||
|
|
||||||
name = "authentik.policies.expiry"
|
name = "authentik.policies.expiry"
|
||||||
label = "authentik_policies_expiry"
|
label = "authentik_policies_expiry"
|
||||||
verbose_name = "authentik Policies.Expiry"
|
verbose_name = "authentik Policies.Expiry"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""Authentik policy_expression app config"""
|
"""Authentik policy_expression app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikPolicyExpressionConfig(AppConfig):
|
class AuthentikPolicyExpressionConfig(ManagedAppConfig):
|
||||||
"""Authentik policy_expression app config"""
|
"""Authentik policy_expression app config"""
|
||||||
|
|
||||||
name = "authentik.policies.expression"
|
name = "authentik.policies.expression"
|
||||||
label = "authentik_policies_expression"
|
label = "authentik_policies_expression"
|
||||||
verbose_name = "authentik Policies.Expression"
|
verbose_name = "authentik Policies.Expression"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""Authentik policy geoip app config"""
|
"""Authentik policy geoip app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikPolicyGeoIPConfig(AppConfig):
|
class AuthentikPolicyGeoIPConfig(ManagedAppConfig):
|
||||||
"""Authentik policy_geoip app config"""
|
"""Authentik policy_geoip app config"""
|
||||||
|
|
||||||
name = "authentik.policies.geoip"
|
name = "authentik.policies.geoip"
|
||||||
label = "authentik_policies_geoip"
|
label = "authentik_policies_geoip"
|
||||||
verbose_name = "authentik Policies.GeoIP"
|
verbose_name = "authentik Policies.GeoIP"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik Password policy app config"""
|
"""authentik Password policy app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikPoliciesPasswordConfig(AppConfig):
|
class AuthentikPoliciesPasswordConfig(ManagedAppConfig):
|
||||||
"""authentik Password policy app config"""
|
"""authentik Password policy app config"""
|
||||||
|
|
||||||
name = "authentik.policies.password"
|
name = "authentik.policies.password"
|
||||||
label = "authentik_policies_password"
|
label = "authentik_policies_password"
|
||||||
verbose_name = "authentik Policies.Password"
|
verbose_name = "authentik Policies.Password"
|
||||||
|
default = True
|
||||||
|
@ -1,9 +1,12 @@
|
|||||||
"""policy engine tests"""
|
"""policy engine tests"""
|
||||||
|
|
||||||
from django.core.cache import cache
|
from django.core.cache import cache
|
||||||
|
from django.db import connections
|
||||||
from django.test import TestCase
|
from django.test import TestCase
|
||||||
|
from django.test.utils import CaptureQueriesContext
|
||||||
|
|
||||||
from authentik.core.tests.utils import create_test_admin_user
|
from authentik.core.models import Group
|
||||||
|
from authentik.core.tests.utils import create_test_user
|
||||||
from authentik.lib.generators import generate_id
|
from authentik.lib.generators import generate_id
|
||||||
from authentik.policies.dummy.models import DummyPolicy
|
from authentik.policies.dummy.models import DummyPolicy
|
||||||
from authentik.policies.engine import PolicyEngine
|
from authentik.policies.engine import PolicyEngine
|
||||||
@ -19,7 +22,7 @@ class TestPolicyEngine(TestCase):
|
|||||||
|
|
||||||
def setUp(self):
|
def setUp(self):
|
||||||
clear_policy_cache()
|
clear_policy_cache()
|
||||||
self.user = create_test_admin_user()
|
self.user = create_test_user()
|
||||||
self.policy_false = DummyPolicy.objects.create(
|
self.policy_false = DummyPolicy.objects.create(
|
||||||
name=generate_id(), result=False, wait_min=0, wait_max=1
|
name=generate_id(), result=False, wait_min=0, wait_max=1
|
||||||
)
|
)
|
||||||
@ -127,3 +130,43 @@ class TestPolicyEngine(TestCase):
|
|||||||
self.assertEqual(len(cache.keys(f"{CACHE_PREFIX}{binding.policy_binding_uuid.hex}*")), 1)
|
self.assertEqual(len(cache.keys(f"{CACHE_PREFIX}{binding.policy_binding_uuid.hex}*")), 1)
|
||||||
self.assertEqual(engine.build().passing, False)
|
self.assertEqual(engine.build().passing, False)
|
||||||
self.assertEqual(len(cache.keys(f"{CACHE_PREFIX}{binding.policy_binding_uuid.hex}*")), 1)
|
self.assertEqual(len(cache.keys(f"{CACHE_PREFIX}{binding.policy_binding_uuid.hex}*")), 1)
|
||||||
|
|
||||||
|
def test_engine_static_bindings(self):
|
||||||
|
"""Test static bindings"""
|
||||||
|
group_a = Group.objects.create(name=generate_id())
|
||||||
|
group_b = Group.objects.create(name=generate_id())
|
||||||
|
group_b.users.add(self.user)
|
||||||
|
user = create_test_user()
|
||||||
|
|
||||||
|
for case in [
|
||||||
|
{
|
||||||
|
"message": "Group, not member",
|
||||||
|
"binding_args": {"group": group_a},
|
||||||
|
"passing": False,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"message": "Group, member",
|
||||||
|
"binding_args": {"group": group_b},
|
||||||
|
"passing": True,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"message": "User, other",
|
||||||
|
"binding_args": {"user": user},
|
||||||
|
"passing": False,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"message": "User, same",
|
||||||
|
"binding_args": {"user": self.user},
|
||||||
|
"passing": True,
|
||||||
|
},
|
||||||
|
]:
|
||||||
|
with self.subTest():
|
||||||
|
pbm = PolicyBindingModel.objects.create()
|
||||||
|
for x in range(1000):
|
||||||
|
PolicyBinding.objects.create(target=pbm, order=x, **case["binding_args"])
|
||||||
|
engine = PolicyEngine(pbm, self.user)
|
||||||
|
engine.use_cache = False
|
||||||
|
with CaptureQueriesContext(connections["default"]) as ctx:
|
||||||
|
engine.build()
|
||||||
|
self.assertLess(ctx.final_queries, 1000)
|
||||||
|
self.assertEqual(engine.result.passing, case["passing"])
|
||||||
|
@ -29,13 +29,12 @@ class TestPolicyProcess(TestCase):
|
|||||||
def setUp(self):
|
def setUp(self):
|
||||||
clear_policy_cache()
|
clear_policy_cache()
|
||||||
self.factory = RequestFactory()
|
self.factory = RequestFactory()
|
||||||
self.user = User.objects.create_user(username="policyuser")
|
self.user = User.objects.create_user(username=generate_id())
|
||||||
|
|
||||||
def test_group_passing(self):
|
def test_group_passing(self):
|
||||||
"""Test binding to group"""
|
"""Test binding to group"""
|
||||||
group = Group.objects.create(name="test-group")
|
group = Group.objects.create(name=generate_id())
|
||||||
group.users.add(self.user)
|
group.users.add(self.user)
|
||||||
group.save()
|
|
||||||
binding = PolicyBinding(group=group)
|
binding = PolicyBinding(group=group)
|
||||||
|
|
||||||
request = PolicyRequest(self.user)
|
request = PolicyRequest(self.user)
|
||||||
@ -44,8 +43,7 @@ class TestPolicyProcess(TestCase):
|
|||||||
|
|
||||||
def test_group_negative(self):
|
def test_group_negative(self):
|
||||||
"""Test binding to group"""
|
"""Test binding to group"""
|
||||||
group = Group.objects.create(name="test-group")
|
group = Group.objects.create(name=generate_id())
|
||||||
group.save()
|
|
||||||
binding = PolicyBinding(group=group)
|
binding = PolicyBinding(group=group)
|
||||||
|
|
||||||
request = PolicyRequest(self.user)
|
request = PolicyRequest(self.user)
|
||||||
@ -115,8 +113,10 @@ class TestPolicyProcess(TestCase):
|
|||||||
|
|
||||||
def test_exception(self):
|
def test_exception(self):
|
||||||
"""Test policy execution"""
|
"""Test policy execution"""
|
||||||
policy = Policy.objects.create(name="test-execution")
|
policy = Policy.objects.create(name=generate_id())
|
||||||
binding = PolicyBinding(policy=policy, target=Application.objects.create(name="test"))
|
binding = PolicyBinding(
|
||||||
|
policy=policy, target=Application.objects.create(name=generate_id())
|
||||||
|
)
|
||||||
|
|
||||||
request = PolicyRequest(self.user)
|
request = PolicyRequest(self.user)
|
||||||
response = PolicyProcess(binding, request, None).execute()
|
response = PolicyProcess(binding, request, None).execute()
|
||||||
@ -125,13 +125,15 @@ class TestPolicyProcess(TestCase):
|
|||||||
def test_execution_logging(self):
|
def test_execution_logging(self):
|
||||||
"""Test policy execution creates event"""
|
"""Test policy execution creates event"""
|
||||||
policy = DummyPolicy.objects.create(
|
policy = DummyPolicy.objects.create(
|
||||||
name="test-execution-logging",
|
name=generate_id(),
|
||||||
result=False,
|
result=False,
|
||||||
wait_min=0,
|
wait_min=0,
|
||||||
wait_max=1,
|
wait_max=1,
|
||||||
execution_logging=True,
|
execution_logging=True,
|
||||||
)
|
)
|
||||||
binding = PolicyBinding(policy=policy, target=Application.objects.create(name="test"))
|
binding = PolicyBinding(
|
||||||
|
policy=policy, target=Application.objects.create(name=generate_id())
|
||||||
|
)
|
||||||
|
|
||||||
http_request = self.factory.get(reverse("authentik_api:user-impersonate-end"))
|
http_request = self.factory.get(reverse("authentik_api:user-impersonate-end"))
|
||||||
http_request.user = self.user
|
http_request.user = self.user
|
||||||
@ -186,13 +188,15 @@ class TestPolicyProcess(TestCase):
|
|||||||
def test_execution_logging_anonymous(self):
|
def test_execution_logging_anonymous(self):
|
||||||
"""Test policy execution creates event with anonymous user"""
|
"""Test policy execution creates event with anonymous user"""
|
||||||
policy = DummyPolicy.objects.create(
|
policy = DummyPolicy.objects.create(
|
||||||
name="test-execution-logging-anon",
|
name=generate_id(),
|
||||||
result=False,
|
result=False,
|
||||||
wait_min=0,
|
wait_min=0,
|
||||||
wait_max=1,
|
wait_max=1,
|
||||||
execution_logging=True,
|
execution_logging=True,
|
||||||
)
|
)
|
||||||
binding = PolicyBinding(policy=policy, target=Application.objects.create(name="test"))
|
binding = PolicyBinding(
|
||||||
|
policy=policy, target=Application.objects.create(name=generate_id())
|
||||||
|
)
|
||||||
|
|
||||||
user = AnonymousUser()
|
user = AnonymousUser()
|
||||||
|
|
||||||
@ -219,9 +223,9 @@ class TestPolicyProcess(TestCase):
|
|||||||
|
|
||||||
def test_raises(self):
|
def test_raises(self):
|
||||||
"""Test policy that raises error"""
|
"""Test policy that raises error"""
|
||||||
policy_raises = ExpressionPolicy.objects.create(name="raises", expression="{{ 0/0 }}")
|
policy_raises = ExpressionPolicy.objects.create(name=generate_id(), expression="{{ 0/0 }}")
|
||||||
binding = PolicyBinding(
|
binding = PolicyBinding(
|
||||||
policy=policy_raises, target=Application.objects.create(name="test")
|
policy=policy_raises, target=Application.objects.create(name=generate_id())
|
||||||
)
|
)
|
||||||
|
|
||||||
request = PolicyRequest(self.user)
|
request = PolicyRequest(self.user)
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik ldap provider app config"""
|
"""authentik ldap provider app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikProviderLDAPConfig(AppConfig):
|
class AuthentikProviderLDAPConfig(ManagedAppConfig):
|
||||||
"""authentik ldap provider app config"""
|
"""authentik ldap provider app config"""
|
||||||
|
|
||||||
name = "authentik.providers.ldap"
|
name = "authentik.providers.ldap"
|
||||||
label = "authentik_providers_ldap"
|
label = "authentik_providers_ldap"
|
||||||
verbose_name = "authentik Providers.LDAP"
|
verbose_name = "authentik Providers.LDAP"
|
||||||
|
default = True
|
||||||
|
@ -10,3 +10,11 @@ class AuthentikProviderProxyConfig(ManagedAppConfig):
|
|||||||
label = "authentik_providers_proxy"
|
label = "authentik_providers_proxy"
|
||||||
verbose_name = "authentik Providers.Proxy"
|
verbose_name = "authentik Providers.Proxy"
|
||||||
default = True
|
default = True
|
||||||
|
|
||||||
|
@ManagedAppConfig.reconcile_tenant
|
||||||
|
def proxy_set_defaults(self):
|
||||||
|
from authentik.providers.proxy.models import ProxyProvider
|
||||||
|
|
||||||
|
for provider in ProxyProvider.objects.all():
|
||||||
|
provider.set_oauth_defaults()
|
||||||
|
provider.save()
|
||||||
|
@ -2,25 +2,13 @@
|
|||||||
|
|
||||||
from asgiref.sync import async_to_sync
|
from asgiref.sync import async_to_sync
|
||||||
from channels.layers import get_channel_layer
|
from channels.layers import get_channel_layer
|
||||||
from django.db import DatabaseError, InternalError, ProgrammingError
|
|
||||||
|
|
||||||
from authentik.outposts.consumer import OUTPOST_GROUP
|
from authentik.outposts.consumer import OUTPOST_GROUP
|
||||||
from authentik.outposts.models import Outpost, OutpostType
|
from authentik.outposts.models import Outpost, OutpostType
|
||||||
from authentik.providers.oauth2.id_token import hash_session_key
|
from authentik.providers.oauth2.id_token import hash_session_key
|
||||||
from authentik.providers.proxy.models import ProxyProvider
|
|
||||||
from authentik.root.celery import CELERY_APP
|
from authentik.root.celery import CELERY_APP
|
||||||
|
|
||||||
|
|
||||||
@CELERY_APP.task(
|
|
||||||
throws=(DatabaseError, ProgrammingError, InternalError),
|
|
||||||
)
|
|
||||||
def proxy_set_defaults():
|
|
||||||
"""Ensure correct defaults are set for all providers"""
|
|
||||||
for provider in ProxyProvider.objects.all():
|
|
||||||
provider.set_oauth_defaults()
|
|
||||||
provider.save()
|
|
||||||
|
|
||||||
|
|
||||||
@CELERY_APP.task()
|
@CELERY_APP.task()
|
||||||
def proxy_on_logout(session_id: str):
|
def proxy_on_logout(session_id: str):
|
||||||
"""Update outpost instances connected to a single outpost"""
|
"""Update outpost instances connected to a single outpost"""
|
||||||
|
@ -166,7 +166,6 @@ class ConnectionToken(ExpiringModel):
|
|||||||
always_merger.merge(settings, default_settings)
|
always_merger.merge(settings, default_settings)
|
||||||
always_merger.merge(settings, self.endpoint.provider.settings)
|
always_merger.merge(settings, self.endpoint.provider.settings)
|
||||||
always_merger.merge(settings, self.endpoint.settings)
|
always_merger.merge(settings, self.endpoint.settings)
|
||||||
always_merger.merge(settings, self.settings)
|
|
||||||
|
|
||||||
def mapping_evaluator(mappings: QuerySet):
|
def mapping_evaluator(mappings: QuerySet):
|
||||||
for mapping in mappings:
|
for mapping in mappings:
|
||||||
@ -191,6 +190,7 @@ class ConnectionToken(ExpiringModel):
|
|||||||
mapping_evaluator(
|
mapping_evaluator(
|
||||||
RACPropertyMapping.objects.filter(endpoint__in=[self.endpoint]).order_by("name")
|
RACPropertyMapping.objects.filter(endpoint__in=[self.endpoint]).order_by("name")
|
||||||
)
|
)
|
||||||
|
always_merger.merge(settings, self.settings)
|
||||||
|
|
||||||
settings["drive-path"] = f"/tmp/connection/{self.token}" # nosec
|
settings["drive-path"] = f"/tmp/connection/{self.token}" # nosec
|
||||||
settings["create-drive-path"] = "true"
|
settings["create-drive-path"] = "true"
|
||||||
|
@ -90,23 +90,6 @@ class TestModels(TransactionTestCase):
|
|||||||
"resize-method": "display-update",
|
"resize-method": "display-update",
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
# Set settings in token
|
|
||||||
token.settings = {
|
|
||||||
"level": "token",
|
|
||||||
}
|
|
||||||
token.save()
|
|
||||||
self.assertEqual(
|
|
||||||
token.get_settings(),
|
|
||||||
{
|
|
||||||
"hostname": self.endpoint.host.split(":")[0],
|
|
||||||
"port": "1324",
|
|
||||||
"client-name": f"authentik - {self.user}",
|
|
||||||
"drive-path": path,
|
|
||||||
"create-drive-path": "true",
|
|
||||||
"level": "token",
|
|
||||||
"resize-method": "display-update",
|
|
||||||
},
|
|
||||||
)
|
|
||||||
# Set settings in property mapping (provider)
|
# Set settings in property mapping (provider)
|
||||||
mapping = RACPropertyMapping.objects.create(
|
mapping = RACPropertyMapping.objects.create(
|
||||||
name=generate_id(),
|
name=generate_id(),
|
||||||
@ -151,3 +134,22 @@ class TestModels(TransactionTestCase):
|
|||||||
"resize-method": "display-update",
|
"resize-method": "display-update",
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
# Set settings in token
|
||||||
|
token.settings = {
|
||||||
|
"level": "token",
|
||||||
|
}
|
||||||
|
token.save()
|
||||||
|
self.assertEqual(
|
||||||
|
token.get_settings(),
|
||||||
|
{
|
||||||
|
"hostname": self.endpoint.host.split(":")[0],
|
||||||
|
"port": "1324",
|
||||||
|
"client-name": f"authentik - {self.user}",
|
||||||
|
"drive-path": path,
|
||||||
|
"create-drive-path": "true",
|
||||||
|
"foo": "true",
|
||||||
|
"bar": "6",
|
||||||
|
"resize-method": "display-update",
|
||||||
|
"level": "token",
|
||||||
|
},
|
||||||
|
)
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik radius provider app config"""
|
"""authentik radius provider app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikProviderRadiusConfig(AppConfig):
|
class AuthentikProviderRadiusConfig(ManagedAppConfig):
|
||||||
"""authentik radius provider app config"""
|
"""authentik radius provider app config"""
|
||||||
|
|
||||||
name = "authentik.providers.radius"
|
name = "authentik.providers.radius"
|
||||||
label = "authentik_providers_radius"
|
label = "authentik_providers_radius"
|
||||||
verbose_name = "authentik Providers.Radius"
|
verbose_name = "authentik Providers.Radius"
|
||||||
|
default = True
|
||||||
|
@ -1,12 +1,13 @@
|
|||||||
"""authentik SAML IdP app config"""
|
"""authentik SAML IdP app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikProviderSAMLConfig(AppConfig):
|
class AuthentikProviderSAMLConfig(ManagedAppConfig):
|
||||||
"""authentik SAML IdP app config"""
|
"""authentik SAML IdP app config"""
|
||||||
|
|
||||||
name = "authentik.providers.saml"
|
name = "authentik.providers.saml"
|
||||||
label = "authentik_providers_saml"
|
label = "authentik_providers_saml"
|
||||||
verbose_name = "authentik Providers.SAML"
|
verbose_name = "authentik Providers.SAML"
|
||||||
mountpoint = "application/saml/"
|
mountpoint = "application/saml/"
|
||||||
|
default = True
|
||||||
|
@ -47,15 +47,16 @@ class SCIMGroupClient(SCIMClient[Group, SCIMProviderGroup, SCIMGroupSchema]):
|
|||||||
|
|
||||||
def to_schema(self, obj: Group, connection: SCIMProviderGroup) -> SCIMGroupSchema:
|
def to_schema(self, obj: Group, connection: SCIMProviderGroup) -> SCIMGroupSchema:
|
||||||
"""Convert authentik user into SCIM"""
|
"""Convert authentik user into SCIM"""
|
||||||
raw_scim_group = super().to_schema(
|
raw_scim_group = super().to_schema(obj, connection)
|
||||||
obj,
|
|
||||||
connection,
|
|
||||||
schemas=(SCIM_GROUP_SCHEMA,),
|
|
||||||
)
|
|
||||||
try:
|
try:
|
||||||
scim_group = SCIMGroupSchema.model_validate(delete_none_values(raw_scim_group))
|
scim_group = SCIMGroupSchema.model_validate(delete_none_values(raw_scim_group))
|
||||||
except ValidationError as exc:
|
except ValidationError as exc:
|
||||||
raise StopSync(exc, obj) from exc
|
raise StopSync(exc, obj) from exc
|
||||||
|
if SCIM_GROUP_SCHEMA not in scim_group.schemas:
|
||||||
|
scim_group.schemas.insert(0, SCIM_GROUP_SCHEMA)
|
||||||
|
# As this might be unset, we need to tell pydantic it's set so ensure the schemas
|
||||||
|
# are included, even if its just the defaults
|
||||||
|
scim_group.schemas = list(scim_group.schemas)
|
||||||
if not scim_group.externalId:
|
if not scim_group.externalId:
|
||||||
scim_group.externalId = str(obj.pk)
|
scim_group.externalId = str(obj.pk)
|
||||||
|
|
||||||
|
@ -31,15 +31,16 @@ class SCIMUserClient(SCIMClient[User, SCIMProviderUser, SCIMUserSchema]):
|
|||||||
|
|
||||||
def to_schema(self, obj: User, connection: SCIMProviderUser) -> SCIMUserSchema:
|
def to_schema(self, obj: User, connection: SCIMProviderUser) -> SCIMUserSchema:
|
||||||
"""Convert authentik user into SCIM"""
|
"""Convert authentik user into SCIM"""
|
||||||
raw_scim_user = super().to_schema(
|
raw_scim_user = super().to_schema(obj, connection)
|
||||||
obj,
|
|
||||||
connection,
|
|
||||||
schemas=(SCIM_USER_SCHEMA,),
|
|
||||||
)
|
|
||||||
try:
|
try:
|
||||||
scim_user = SCIMUserSchema.model_validate(delete_none_values(raw_scim_user))
|
scim_user = SCIMUserSchema.model_validate(delete_none_values(raw_scim_user))
|
||||||
except ValidationError as exc:
|
except ValidationError as exc:
|
||||||
raise StopSync(exc, obj) from exc
|
raise StopSync(exc, obj) from exc
|
||||||
|
if SCIM_USER_SCHEMA not in scim_user.schemas:
|
||||||
|
scim_user.schemas.insert(0, SCIM_USER_SCHEMA)
|
||||||
|
# As this might be unset, we need to tell pydantic it's set so ensure the schemas
|
||||||
|
# are included, even if its just the defaults
|
||||||
|
scim_user.schemas = list(scim_user.schemas)
|
||||||
if not scim_user.externalId:
|
if not scim_user.externalId:
|
||||||
scim_user.externalId = str(obj.uid)
|
scim_user.externalId = str(obj.uid)
|
||||||
return scim_user
|
return scim_user
|
||||||
|
@ -91,6 +91,57 @@ class SCIMUserTests(TestCase):
|
|||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
|
||||||
|
@Mocker()
|
||||||
|
def test_user_create_custom_schema(self, mock: Mocker):
|
||||||
|
"""Test user creation with custom schema"""
|
||||||
|
schema = SCIMMapping.objects.create(
|
||||||
|
name="custom_schema",
|
||||||
|
expression="""return {"schemas": ["foo"]}""",
|
||||||
|
)
|
||||||
|
self.provider.property_mappings.add(schema)
|
||||||
|
scim_id = generate_id()
|
||||||
|
mock.get(
|
||||||
|
"https://localhost/ServiceProviderConfig",
|
||||||
|
json={},
|
||||||
|
)
|
||||||
|
mock.post(
|
||||||
|
"https://localhost/Users",
|
||||||
|
json={
|
||||||
|
"id": scim_id,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
uid = generate_id()
|
||||||
|
user = User.objects.create(
|
||||||
|
username=uid,
|
||||||
|
name=f"{uid} {uid}",
|
||||||
|
email=f"{uid}@goauthentik.io",
|
||||||
|
)
|
||||||
|
self.assertEqual(mock.call_count, 2)
|
||||||
|
self.assertEqual(mock.request_history[0].method, "GET")
|
||||||
|
self.assertEqual(mock.request_history[1].method, "POST")
|
||||||
|
self.assertJSONEqual(
|
||||||
|
mock.request_history[1].body,
|
||||||
|
{
|
||||||
|
"schemas": ["urn:ietf:params:scim:schemas:core:2.0:User", "foo"],
|
||||||
|
"active": True,
|
||||||
|
"emails": [
|
||||||
|
{
|
||||||
|
"primary": True,
|
||||||
|
"type": "other",
|
||||||
|
"value": f"{uid}@goauthentik.io",
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"externalId": user.uid,
|
||||||
|
"name": {
|
||||||
|
"familyName": uid,
|
||||||
|
"formatted": f"{uid} {uid}",
|
||||||
|
"givenName": uid,
|
||||||
|
},
|
||||||
|
"displayName": f"{uid} {uid}",
|
||||||
|
"userName": uid,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
|
||||||
@Mocker()
|
@Mocker()
|
||||||
def test_user_create_different_provider_same_id(self, mock: Mocker):
|
def test_user_create_different_provider_same_id(self, mock: Mocker):
|
||||||
"""Test user creation with multiple providers that happen
|
"""Test user creation with multiple providers that happen
|
||||||
@ -384,7 +435,7 @@ class SCIMUserTests(TestCase):
|
|||||||
self.assertIn(request.method, SAFE_METHODS)
|
self.assertIn(request.method, SAFE_METHODS)
|
||||||
task = SystemTask.objects.filter(uid=slugify(self.provider.name)).first()
|
task = SystemTask.objects.filter(uid=slugify(self.provider.name)).first()
|
||||||
self.assertIsNotNone(task)
|
self.assertIsNotNone(task)
|
||||||
drop_msg = task.messages[2]
|
drop_msg = task.messages[3]
|
||||||
self.assertEqual(drop_msg["event"], "Dropping mutating request due to dry run")
|
self.assertEqual(drop_msg["event"], "Dropping mutating request due to dry run")
|
||||||
self.assertIsNotNone(drop_msg["attributes"]["url"])
|
self.assertIsNotNone(drop_msg["attributes"]["url"])
|
||||||
self.assertIsNotNone(drop_msg["attributes"]["body"])
|
self.assertIsNotNone(drop_msg["attributes"]["body"])
|
||||||
|
@ -1,12 +1,13 @@
|
|||||||
"""authentik Recovery app config"""
|
"""authentik Recovery app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikRecoveryConfig(AppConfig):
|
class AuthentikRecoveryConfig(ManagedAppConfig):
|
||||||
"""authentik Recovery app config"""
|
"""authentik Recovery app config"""
|
||||||
|
|
||||||
name = "authentik.recovery"
|
name = "authentik.recovery"
|
||||||
label = "authentik_recovery"
|
label = "authentik_recovery"
|
||||||
verbose_name = "authentik Recovery"
|
verbose_name = "authentik Recovery"
|
||||||
mountpoint = "recovery/"
|
mountpoint = "recovery/"
|
||||||
|
default = True
|
||||||
|
@ -98,13 +98,7 @@ def _get_startup_tasks_default_tenant() -> list[Callable]:
|
|||||||
|
|
||||||
def _get_startup_tasks_all_tenants() -> list[Callable]:
|
def _get_startup_tasks_all_tenants() -> list[Callable]:
|
||||||
"""Get all tasks to be run on startup for all tenants"""
|
"""Get all tasks to be run on startup for all tenants"""
|
||||||
from authentik.admin.tasks import clear_update_notifications
|
return []
|
||||||
from authentik.providers.proxy.tasks import proxy_set_defaults
|
|
||||||
|
|
||||||
return [
|
|
||||||
clear_update_notifications,
|
|
||||||
proxy_set_defaults,
|
|
||||||
]
|
|
||||||
|
|
||||||
|
|
||||||
@worker_ready.connect
|
@worker_ready.connect
|
||||||
|
@ -424,7 +424,7 @@ else:
|
|||||||
"BACKEND": "authentik.root.storages.FileStorage",
|
"BACKEND": "authentik.root.storages.FileStorage",
|
||||||
"OPTIONS": {
|
"OPTIONS": {
|
||||||
"location": Path(CONFIG.get("storage.media.file.path")),
|
"location": Path(CONFIG.get("storage.media.file.path")),
|
||||||
"base_url": "/media/",
|
"base_url": CONFIG.get("web.path", "/") + "media/",
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
# Compatibility for apps not supporting top-level STORAGES
|
# Compatibility for apps not supporting top-level STORAGES
|
||||||
|
@ -31,6 +31,8 @@ class PytestTestRunner(DiscoverRunner): # pragma: no cover
|
|||||||
|
|
||||||
if kwargs.get("randomly_seed", None):
|
if kwargs.get("randomly_seed", None):
|
||||||
self.args.append(f"--randomly-seed={kwargs['randomly_seed']}")
|
self.args.append(f"--randomly-seed={kwargs['randomly_seed']}")
|
||||||
|
if kwargs.get("no_capture", False):
|
||||||
|
self.args.append("--capture=no")
|
||||||
|
|
||||||
settings.TEST = True
|
settings.TEST = True
|
||||||
settings.CELERY["task_always_eager"] = True
|
settings.CELERY["task_always_eager"] = True
|
||||||
@ -64,6 +66,11 @@ class PytestTestRunner(DiscoverRunner): # pragma: no cover
|
|||||||
"Default behaviour: use random.Random().getrandbits(32), so the seed is"
|
"Default behaviour: use random.Random().getrandbits(32), so the seed is"
|
||||||
"different on each run.",
|
"different on each run.",
|
||||||
)
|
)
|
||||||
|
parser.add_argument(
|
||||||
|
"--no-capture",
|
||||||
|
action="store_true",
|
||||||
|
help="Disable any capturing of stdout/stderr during tests.",
|
||||||
|
)
|
||||||
|
|
||||||
def run_tests(self, test_labels, extra_tests=None, **kwargs):
|
def run_tests(self, test_labels, extra_tests=None, **kwargs):
|
||||||
"""Run pytest and return the exitcode.
|
"""Run pytest and return the exitcode.
|
||||||
|
@ -103,6 +103,7 @@ class LDAPSourceSerializer(SourceSerializer):
|
|||||||
"user_object_filter",
|
"user_object_filter",
|
||||||
"group_object_filter",
|
"group_object_filter",
|
||||||
"group_membership_field",
|
"group_membership_field",
|
||||||
|
"user_membership_attribute",
|
||||||
"object_uniqueness_field",
|
"object_uniqueness_field",
|
||||||
"password_login_update_internal_password",
|
"password_login_update_internal_password",
|
||||||
"sync_users",
|
"sync_users",
|
||||||
@ -111,6 +112,7 @@ class LDAPSourceSerializer(SourceSerializer):
|
|||||||
"sync_parent_group",
|
"sync_parent_group",
|
||||||
"connectivity",
|
"connectivity",
|
||||||
"lookup_groups_from_user",
|
"lookup_groups_from_user",
|
||||||
|
"delete_not_found_objects",
|
||||||
]
|
]
|
||||||
extra_kwargs = {"bind_password": {"write_only": True}}
|
extra_kwargs = {"bind_password": {"write_only": True}}
|
||||||
|
|
||||||
@ -138,6 +140,7 @@ class LDAPSourceViewSet(UsedByMixin, ModelViewSet):
|
|||||||
"user_object_filter",
|
"user_object_filter",
|
||||||
"group_object_filter",
|
"group_object_filter",
|
||||||
"group_membership_field",
|
"group_membership_field",
|
||||||
|
"user_membership_attribute",
|
||||||
"object_uniqueness_field",
|
"object_uniqueness_field",
|
||||||
"password_login_update_internal_password",
|
"password_login_update_internal_password",
|
||||||
"sync_users",
|
"sync_users",
|
||||||
@ -147,6 +150,7 @@ class LDAPSourceViewSet(UsedByMixin, ModelViewSet):
|
|||||||
"user_property_mappings",
|
"user_property_mappings",
|
||||||
"group_property_mappings",
|
"group_property_mappings",
|
||||||
"lookup_groups_from_user",
|
"lookup_groups_from_user",
|
||||||
|
"delete_not_found_objects",
|
||||||
]
|
]
|
||||||
search_fields = ["name", "slug"]
|
search_fields = ["name", "slug"]
|
||||||
ordering = ["name"]
|
ordering = ["name"]
|
||||||
|
@ -0,0 +1,48 @@
|
|||||||
|
# Generated by Django 5.1.9 on 2025-05-28 08:15
|
||||||
|
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
|
dependencies = [
|
||||||
|
("authentik_core", "0048_delete_oldauthenticatedsession_content_type"),
|
||||||
|
("authentik_sources_ldap", "0008_groupldapsourceconnection_userldapsourceconnection"),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AddField(
|
||||||
|
model_name="groupldapsourceconnection",
|
||||||
|
name="validated_by",
|
||||||
|
field=models.UUIDField(
|
||||||
|
blank=True,
|
||||||
|
help_text="Unique ID used while checking if this object still exists in the directory.",
|
||||||
|
null=True,
|
||||||
|
),
|
||||||
|
),
|
||||||
|
migrations.AddField(
|
||||||
|
model_name="ldapsource",
|
||||||
|
name="delete_not_found_objects",
|
||||||
|
field=models.BooleanField(
|
||||||
|
default=False,
|
||||||
|
help_text="Delete authentik users and groups which were previously supplied by this source, but are now missing from it.",
|
||||||
|
),
|
||||||
|
),
|
||||||
|
migrations.AddField(
|
||||||
|
model_name="userldapsourceconnection",
|
||||||
|
name="validated_by",
|
||||||
|
field=models.UUIDField(
|
||||||
|
blank=True,
|
||||||
|
help_text="Unique ID used while checking if this object still exists in the directory.",
|
||||||
|
null=True,
|
||||||
|
),
|
||||||
|
),
|
||||||
|
migrations.AddIndex(
|
||||||
|
model_name="groupldapsourceconnection",
|
||||||
|
index=models.Index(fields=["validated_by"], name="authentik_s_validat_b70447_idx"),
|
||||||
|
),
|
||||||
|
migrations.AddIndex(
|
||||||
|
model_name="userldapsourceconnection",
|
||||||
|
index=models.Index(fields=["validated_by"], name="authentik_s_validat_ff2ebc_idx"),
|
||||||
|
),
|
||||||
|
]
|
@ -0,0 +1,32 @@
|
|||||||
|
# Generated by Django 5.1.9 on 2025-05-29 11:22
|
||||||
|
|
||||||
|
from django.apps.registry import Apps
|
||||||
|
from django.db import migrations, models
|
||||||
|
from django.db.backends.base.schema import BaseDatabaseSchemaEditor
|
||||||
|
|
||||||
|
|
||||||
|
def set_user_membership_attribute(apps: Apps, schema_editor: BaseDatabaseSchemaEditor):
|
||||||
|
LDAPSource = apps.get_model("authentik_sources_ldap", "LDAPSource")
|
||||||
|
db_alias = schema_editor.connection.alias
|
||||||
|
|
||||||
|
LDAPSource.objects.using(db_alias).filter(group_membership_field="memberUid").all().update(
|
||||||
|
user_membership_attribute="ldap_uniq"
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
dependencies = [
|
||||||
|
("authentik_sources_ldap", "0009_groupldapsourceconnection_validated_by_and_more"),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AddField(
|
||||||
|
model_name="ldapsource",
|
||||||
|
name="user_membership_attribute",
|
||||||
|
field=models.TextField(
|
||||||
|
default="distinguishedName",
|
||||||
|
help_text="Attribute which matches the value of `group_membership_field`.",
|
||||||
|
),
|
||||||
|
),
|
||||||
|
migrations.RunPython(set_user_membership_attribute, migrations.RunPython.noop),
|
||||||
|
]
|
@ -100,6 +100,10 @@ class LDAPSource(Source):
|
|||||||
default="(objectClass=person)",
|
default="(objectClass=person)",
|
||||||
help_text=_("Consider Objects matching this filter to be Users."),
|
help_text=_("Consider Objects matching this filter to be Users."),
|
||||||
)
|
)
|
||||||
|
user_membership_attribute = models.TextField(
|
||||||
|
default=LDAP_DISTINGUISHED_NAME,
|
||||||
|
help_text=_("Attribute which matches the value of `group_membership_field`."),
|
||||||
|
)
|
||||||
group_membership_field = models.TextField(
|
group_membership_field = models.TextField(
|
||||||
default="member", help_text=_("Field which contains members of a group.")
|
default="member", help_text=_("Field which contains members of a group.")
|
||||||
)
|
)
|
||||||
@ -137,6 +141,14 @@ class LDAPSource(Source):
|
|||||||
),
|
),
|
||||||
)
|
)
|
||||||
|
|
||||||
|
delete_not_found_objects = models.BooleanField(
|
||||||
|
default=False,
|
||||||
|
help_text=_(
|
||||||
|
"Delete authentik users and groups which were previously supplied by this source, "
|
||||||
|
"but are now missing from it."
|
||||||
|
),
|
||||||
|
)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def component(self) -> str:
|
def component(self) -> str:
|
||||||
return "ak-source-ldap-form"
|
return "ak-source-ldap-form"
|
||||||
@ -321,6 +333,12 @@ class LDAPSourcePropertyMapping(PropertyMapping):
|
|||||||
|
|
||||||
|
|
||||||
class UserLDAPSourceConnection(UserSourceConnection):
|
class UserLDAPSourceConnection(UserSourceConnection):
|
||||||
|
validated_by = models.UUIDField(
|
||||||
|
null=True,
|
||||||
|
blank=True,
|
||||||
|
help_text=_("Unique ID used while checking if this object still exists in the directory."),
|
||||||
|
)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def serializer(self) -> type[Serializer]:
|
def serializer(self) -> type[Serializer]:
|
||||||
from authentik.sources.ldap.api import (
|
from authentik.sources.ldap.api import (
|
||||||
@ -332,9 +350,18 @@ class UserLDAPSourceConnection(UserSourceConnection):
|
|||||||
class Meta:
|
class Meta:
|
||||||
verbose_name = _("User LDAP Source Connection")
|
verbose_name = _("User LDAP Source Connection")
|
||||||
verbose_name_plural = _("User LDAP Source Connections")
|
verbose_name_plural = _("User LDAP Source Connections")
|
||||||
|
indexes = [
|
||||||
|
models.Index(fields=["validated_by"]),
|
||||||
|
]
|
||||||
|
|
||||||
|
|
||||||
class GroupLDAPSourceConnection(GroupSourceConnection):
|
class GroupLDAPSourceConnection(GroupSourceConnection):
|
||||||
|
validated_by = models.UUIDField(
|
||||||
|
null=True,
|
||||||
|
blank=True,
|
||||||
|
help_text=_("Unique ID used while checking if this object still exists in the directory."),
|
||||||
|
)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def serializer(self) -> type[Serializer]:
|
def serializer(self) -> type[Serializer]:
|
||||||
from authentik.sources.ldap.api import (
|
from authentik.sources.ldap.api import (
|
||||||
@ -346,3 +373,6 @@ class GroupLDAPSourceConnection(GroupSourceConnection):
|
|||||||
class Meta:
|
class Meta:
|
||||||
verbose_name = _("Group LDAP Source Connection")
|
verbose_name = _("Group LDAP Source Connection")
|
||||||
verbose_name_plural = _("Group LDAP Source Connections")
|
verbose_name_plural = _("Group LDAP Source Connections")
|
||||||
|
indexes = [
|
||||||
|
models.Index(fields=["validated_by"]),
|
||||||
|
]
|
||||||
|
@ -9,7 +9,7 @@ from structlog.stdlib import BoundLogger, get_logger
|
|||||||
from authentik.core.sources.mapper import SourceMapper
|
from authentik.core.sources.mapper import SourceMapper
|
||||||
from authentik.lib.config import CONFIG
|
from authentik.lib.config import CONFIG
|
||||||
from authentik.lib.sync.mapper import PropertyMappingManager
|
from authentik.lib.sync.mapper import PropertyMappingManager
|
||||||
from authentik.sources.ldap.models import LDAPSource
|
from authentik.sources.ldap.models import LDAPSource, flatten
|
||||||
|
|
||||||
|
|
||||||
class BaseLDAPSynchronizer:
|
class BaseLDAPSynchronizer:
|
||||||
@ -77,6 +77,16 @@ class BaseLDAPSynchronizer:
|
|||||||
"""Get objects from LDAP, implemented in subclass"""
|
"""Get objects from LDAP, implemented in subclass"""
|
||||||
raise NotImplementedError()
|
raise NotImplementedError()
|
||||||
|
|
||||||
|
def get_attributes(self, object):
|
||||||
|
if "attributes" not in object:
|
||||||
|
return
|
||||||
|
return object.get("attributes", {})
|
||||||
|
|
||||||
|
def get_identifier(self, attributes: dict):
|
||||||
|
if not attributes.get(self._source.object_uniqueness_field):
|
||||||
|
return
|
||||||
|
return flatten(attributes[self._source.object_uniqueness_field])
|
||||||
|
|
||||||
def search_paginator( # noqa: PLR0913
|
def search_paginator( # noqa: PLR0913
|
||||||
self,
|
self,
|
||||||
search_base,
|
search_base,
|
||||||
|
61
authentik/sources/ldap/sync/forward_delete_groups.py
Normal file
61
authentik/sources/ldap/sync/forward_delete_groups.py
Normal file
@ -0,0 +1,61 @@
|
|||||||
|
from collections.abc import Generator
|
||||||
|
from itertools import batched
|
||||||
|
from uuid import uuid4
|
||||||
|
|
||||||
|
from ldap3 import SUBTREE
|
||||||
|
|
||||||
|
from authentik.core.models import Group
|
||||||
|
from authentik.sources.ldap.models import GroupLDAPSourceConnection
|
||||||
|
from authentik.sources.ldap.sync.base import BaseLDAPSynchronizer
|
||||||
|
from authentik.sources.ldap.sync.forward_delete_users import DELETE_CHUNK_SIZE, UPDATE_CHUNK_SIZE
|
||||||
|
|
||||||
|
|
||||||
|
class GroupLDAPForwardDeletion(BaseLDAPSynchronizer):
|
||||||
|
"""Delete LDAP Groups from authentik"""
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
def name() -> str:
|
||||||
|
return "group_deletions"
|
||||||
|
|
||||||
|
def get_objects(self, **kwargs) -> Generator:
|
||||||
|
if not self._source.sync_groups or not self._source.delete_not_found_objects:
|
||||||
|
self.message("Group syncing is disabled for this Source")
|
||||||
|
return iter(())
|
||||||
|
|
||||||
|
uuid = uuid4()
|
||||||
|
groups = self._source.connection().extend.standard.paged_search(
|
||||||
|
search_base=self.base_dn_groups,
|
||||||
|
search_filter=self._source.group_object_filter,
|
||||||
|
search_scope=SUBTREE,
|
||||||
|
attributes=[self._source.object_uniqueness_field],
|
||||||
|
generator=True,
|
||||||
|
**kwargs,
|
||||||
|
)
|
||||||
|
for batch in batched(groups, UPDATE_CHUNK_SIZE, strict=False):
|
||||||
|
identifiers = []
|
||||||
|
for group in batch:
|
||||||
|
if not (attributes := self.get_attributes(group)):
|
||||||
|
continue
|
||||||
|
if identifier := self.get_identifier(attributes):
|
||||||
|
identifiers.append(identifier)
|
||||||
|
GroupLDAPSourceConnection.objects.filter(identifier__in=identifiers).update(
|
||||||
|
validated_by=uuid
|
||||||
|
)
|
||||||
|
|
||||||
|
return batched(
|
||||||
|
GroupLDAPSourceConnection.objects.filter(source=self._source)
|
||||||
|
.exclude(validated_by=uuid)
|
||||||
|
.values_list("group", flat=True)
|
||||||
|
.iterator(chunk_size=DELETE_CHUNK_SIZE),
|
||||||
|
DELETE_CHUNK_SIZE,
|
||||||
|
strict=False,
|
||||||
|
)
|
||||||
|
|
||||||
|
def sync(self, group_pks: tuple) -> int:
|
||||||
|
"""Delete authentik groups"""
|
||||||
|
if not self._source.sync_groups or not self._source.delete_not_found_objects:
|
||||||
|
self.message("Group syncing is disabled for this Source")
|
||||||
|
return -1
|
||||||
|
self._logger.debug("Deleting groups", group_pks=group_pks)
|
||||||
|
_, deleted_per_type = Group.objects.filter(pk__in=group_pks).delete()
|
||||||
|
return deleted_per_type.get(Group._meta.label, 0)
|
63
authentik/sources/ldap/sync/forward_delete_users.py
Normal file
63
authentik/sources/ldap/sync/forward_delete_users.py
Normal file
@ -0,0 +1,63 @@
|
|||||||
|
from collections.abc import Generator
|
||||||
|
from itertools import batched
|
||||||
|
from uuid import uuid4
|
||||||
|
|
||||||
|
from ldap3 import SUBTREE
|
||||||
|
|
||||||
|
from authentik.core.models import User
|
||||||
|
from authentik.sources.ldap.models import UserLDAPSourceConnection
|
||||||
|
from authentik.sources.ldap.sync.base import BaseLDAPSynchronizer
|
||||||
|
|
||||||
|
UPDATE_CHUNK_SIZE = 10_000
|
||||||
|
DELETE_CHUNK_SIZE = 50
|
||||||
|
|
||||||
|
|
||||||
|
class UserLDAPForwardDeletion(BaseLDAPSynchronizer):
|
||||||
|
"""Delete LDAP Users from authentik"""
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
def name() -> str:
|
||||||
|
return "user_deletions"
|
||||||
|
|
||||||
|
def get_objects(self, **kwargs) -> Generator:
|
||||||
|
if not self._source.sync_users or not self._source.delete_not_found_objects:
|
||||||
|
self.message("User syncing is disabled for this Source")
|
||||||
|
return iter(())
|
||||||
|
|
||||||
|
uuid = uuid4()
|
||||||
|
users = self._source.connection().extend.standard.paged_search(
|
||||||
|
search_base=self.base_dn_users,
|
||||||
|
search_filter=self._source.user_object_filter,
|
||||||
|
search_scope=SUBTREE,
|
||||||
|
attributes=[self._source.object_uniqueness_field],
|
||||||
|
generator=True,
|
||||||
|
**kwargs,
|
||||||
|
)
|
||||||
|
for batch in batched(users, UPDATE_CHUNK_SIZE, strict=False):
|
||||||
|
identifiers = []
|
||||||
|
for user in batch:
|
||||||
|
if not (attributes := self.get_attributes(user)):
|
||||||
|
continue
|
||||||
|
if identifier := self.get_identifier(attributes):
|
||||||
|
identifiers.append(identifier)
|
||||||
|
UserLDAPSourceConnection.objects.filter(identifier__in=identifiers).update(
|
||||||
|
validated_by=uuid
|
||||||
|
)
|
||||||
|
|
||||||
|
return batched(
|
||||||
|
UserLDAPSourceConnection.objects.filter(source=self._source)
|
||||||
|
.exclude(validated_by=uuid)
|
||||||
|
.values_list("user", flat=True)
|
||||||
|
.iterator(chunk_size=DELETE_CHUNK_SIZE),
|
||||||
|
DELETE_CHUNK_SIZE,
|
||||||
|
strict=False,
|
||||||
|
)
|
||||||
|
|
||||||
|
def sync(self, user_pks: tuple) -> int:
|
||||||
|
"""Delete authentik users"""
|
||||||
|
if not self._source.sync_users or not self._source.delete_not_found_objects:
|
||||||
|
self.message("User syncing is disabled for this Source")
|
||||||
|
return -1
|
||||||
|
self._logger.debug("Deleting users", user_pks=user_pks)
|
||||||
|
_, deleted_per_type = User.objects.filter(pk__in=user_pks).delete()
|
||||||
|
return deleted_per_type.get(User._meta.label, 0)
|
@ -58,18 +58,16 @@ class GroupLDAPSynchronizer(BaseLDAPSynchronizer):
|
|||||||
return -1
|
return -1
|
||||||
group_count = 0
|
group_count = 0
|
||||||
for group in page_data:
|
for group in page_data:
|
||||||
if "attributes" not in group:
|
if (attributes := self.get_attributes(group)) is None:
|
||||||
continue
|
continue
|
||||||
attributes = group.get("attributes", {})
|
|
||||||
group_dn = flatten(flatten(group.get("entryDN", group.get("dn"))))
|
group_dn = flatten(flatten(group.get("entryDN", group.get("dn"))))
|
||||||
if not attributes.get(self._source.object_uniqueness_field):
|
if not (uniq := self.get_identifier(attributes)):
|
||||||
self.message(
|
self.message(
|
||||||
f"Uniqueness field not found/not set in attributes: '{group_dn}'",
|
f"Uniqueness field not found/not set in attributes: '{group_dn}'",
|
||||||
attributes=attributes.keys(),
|
attributes=attributes.keys(),
|
||||||
dn=group_dn,
|
dn=group_dn,
|
||||||
)
|
)
|
||||||
continue
|
continue
|
||||||
uniq = flatten(attributes[self._source.object_uniqueness_field])
|
|
||||||
try:
|
try:
|
||||||
defaults = {
|
defaults = {
|
||||||
k: flatten(v)
|
k: flatten(v)
|
||||||
|
@ -63,25 +63,19 @@ class MembershipLDAPSynchronizer(BaseLDAPSynchronizer):
|
|||||||
group_member_dn = group_member.get("dn", {})
|
group_member_dn = group_member.get("dn", {})
|
||||||
members.append(group_member_dn)
|
members.append(group_member_dn)
|
||||||
else:
|
else:
|
||||||
if "attributes" not in group:
|
if (attributes := self.get_attributes(group)) is None:
|
||||||
continue
|
continue
|
||||||
members = group.get("attributes", {}).get(self._source.group_membership_field, [])
|
members = attributes.get(self._source.group_membership_field, [])
|
||||||
|
|
||||||
ak_group = self.get_group(group)
|
ak_group = self.get_group(group)
|
||||||
if not ak_group:
|
if not ak_group:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
membership_mapping_attribute = LDAP_DISTINGUISHED_NAME
|
|
||||||
if self._source.group_membership_field == "memberUid":
|
|
||||||
# If memberships are based on the posixGroup's 'memberUid'
|
|
||||||
# attribute we use the RDN instead of the FDN to lookup members.
|
|
||||||
membership_mapping_attribute = LDAP_UNIQUENESS
|
|
||||||
|
|
||||||
users = User.objects.filter(
|
users = User.objects.filter(
|
||||||
Q(**{f"attributes__{membership_mapping_attribute}__in": members})
|
Q(**{f"attributes__{self._source.user_membership_attribute}__in": members})
|
||||||
| Q(
|
| Q(
|
||||||
**{
|
**{
|
||||||
f"attributes__{membership_mapping_attribute}__isnull": True,
|
f"attributes__{self._source.user_membership_attribute}__isnull": True,
|
||||||
"ak_groups__in": [ak_group],
|
"ak_groups__in": [ak_group],
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
|
@ -60,18 +60,16 @@ class UserLDAPSynchronizer(BaseLDAPSynchronizer):
|
|||||||
return -1
|
return -1
|
||||||
user_count = 0
|
user_count = 0
|
||||||
for user in page_data:
|
for user in page_data:
|
||||||
if "attributes" not in user:
|
if (attributes := self.get_attributes(user)) is None:
|
||||||
continue
|
continue
|
||||||
attributes = user.get("attributes", {})
|
|
||||||
user_dn = flatten(user.get("entryDN", user.get("dn")))
|
user_dn = flatten(user.get("entryDN", user.get("dn")))
|
||||||
if not attributes.get(self._source.object_uniqueness_field):
|
if not (uniq := self.get_identifier(attributes)):
|
||||||
self.message(
|
self.message(
|
||||||
f"Uniqueness field not found/not set in attributes: '{user_dn}'",
|
f"Uniqueness field not found/not set in attributes: '{user_dn}'",
|
||||||
attributes=attributes.keys(),
|
attributes=attributes.keys(),
|
||||||
dn=user_dn,
|
dn=user_dn,
|
||||||
)
|
)
|
||||||
continue
|
continue
|
||||||
uniq = flatten(attributes[self._source.object_uniqueness_field])
|
|
||||||
try:
|
try:
|
||||||
defaults = {
|
defaults = {
|
||||||
k: flatten(v)
|
k: flatten(v)
|
||||||
|
@ -17,6 +17,8 @@ from authentik.lib.utils.reflection import class_to_path, path_to_class
|
|||||||
from authentik.root.celery import CELERY_APP
|
from authentik.root.celery import CELERY_APP
|
||||||
from authentik.sources.ldap.models import LDAPSource
|
from authentik.sources.ldap.models import LDAPSource
|
||||||
from authentik.sources.ldap.sync.base import BaseLDAPSynchronizer
|
from authentik.sources.ldap.sync.base import BaseLDAPSynchronizer
|
||||||
|
from authentik.sources.ldap.sync.forward_delete_groups import GroupLDAPForwardDeletion
|
||||||
|
from authentik.sources.ldap.sync.forward_delete_users import UserLDAPForwardDeletion
|
||||||
from authentik.sources.ldap.sync.groups import GroupLDAPSynchronizer
|
from authentik.sources.ldap.sync.groups import GroupLDAPSynchronizer
|
||||||
from authentik.sources.ldap.sync.membership import MembershipLDAPSynchronizer
|
from authentik.sources.ldap.sync.membership import MembershipLDAPSynchronizer
|
||||||
from authentik.sources.ldap.sync.users import UserLDAPSynchronizer
|
from authentik.sources.ldap.sync.users import UserLDAPSynchronizer
|
||||||
@ -52,11 +54,11 @@ def ldap_connectivity_check(pk: str | None = None):
|
|||||||
|
|
||||||
|
|
||||||
@CELERY_APP.task(
|
@CELERY_APP.task(
|
||||||
# We take the configured hours timeout time by 2.5 as we run user and
|
# We take the configured hours timeout time by 3.5 as we run user and
|
||||||
# group in parallel and then membership, so 2x is to cover the serial tasks,
|
# group in parallel and then membership, then deletions, so 3x is to cover the serial tasks,
|
||||||
# and 0.5x on top of that to give some more leeway
|
# and 0.5x on top of that to give some more leeway
|
||||||
soft_time_limit=(60 * 60 * CONFIG.get_int("ldap.task_timeout_hours")) * 2.5,
|
soft_time_limit=(60 * 60 * CONFIG.get_int("ldap.task_timeout_hours")) * 3.5,
|
||||||
task_time_limit=(60 * 60 * CONFIG.get_int("ldap.task_timeout_hours")) * 2.5,
|
task_time_limit=(60 * 60 * CONFIG.get_int("ldap.task_timeout_hours")) * 3.5,
|
||||||
)
|
)
|
||||||
def ldap_sync_single(source_pk: str):
|
def ldap_sync_single(source_pk: str):
|
||||||
"""Sync a single source"""
|
"""Sync a single source"""
|
||||||
@ -79,6 +81,25 @@ def ldap_sync_single(source_pk: str):
|
|||||||
group(
|
group(
|
||||||
ldap_sync_paginator(source, MembershipLDAPSynchronizer),
|
ldap_sync_paginator(source, MembershipLDAPSynchronizer),
|
||||||
),
|
),
|
||||||
|
# Finally, deletions. What we'd really like to do here is something like
|
||||||
|
# ```
|
||||||
|
# user_identifiers = <ldap query>
|
||||||
|
# User.objects.exclude(
|
||||||
|
# usersourceconnection__identifier__in=user_uniqueness_identifiers,
|
||||||
|
# ).delete()
|
||||||
|
# ```
|
||||||
|
# This runs into performance issues in large installations. So instead we spread the
|
||||||
|
# work out into three steps:
|
||||||
|
# 1. Get every object from the LDAP source.
|
||||||
|
# 2. Mark every object as "safe" in the database. This is quick, but any error could
|
||||||
|
# mean deleting users which should not be deleted, so we do it immediately, in
|
||||||
|
# large chunks, and only queue the deletion step afterwards.
|
||||||
|
# 3. Delete every unmarked item. This is slow, so we spread it over many tasks in
|
||||||
|
# small chunks.
|
||||||
|
group(
|
||||||
|
ldap_sync_paginator(source, UserLDAPForwardDeletion)
|
||||||
|
+ ldap_sync_paginator(source, GroupLDAPForwardDeletion),
|
||||||
|
),
|
||||||
)
|
)
|
||||||
task()
|
task()
|
||||||
|
|
||||||
|
@ -2,6 +2,33 @@
|
|||||||
|
|
||||||
from ldap3 import MOCK_SYNC, OFFLINE_SLAPD_2_4, Connection, Server
|
from ldap3 import MOCK_SYNC, OFFLINE_SLAPD_2_4, Connection, Server
|
||||||
|
|
||||||
|
# The mock modifies these in place, so we have to define them per string
|
||||||
|
user_in_slapd_dn = "cn=user_in_slapd_cn,ou=users,dc=goauthentik,dc=io"
|
||||||
|
user_in_slapd_cn = "user_in_slapd_cn"
|
||||||
|
user_in_slapd_uid = "user_in_slapd_uid"
|
||||||
|
user_in_slapd_object_class = "person"
|
||||||
|
user_in_slapd = {
|
||||||
|
"dn": user_in_slapd_dn,
|
||||||
|
"attributes": {
|
||||||
|
"cn": user_in_slapd_cn,
|
||||||
|
"uid": user_in_slapd_uid,
|
||||||
|
"objectClass": user_in_slapd_object_class,
|
||||||
|
},
|
||||||
|
}
|
||||||
|
group_in_slapd_dn = "cn=user_in_slapd_cn,ou=groups,dc=goauthentik,dc=io"
|
||||||
|
group_in_slapd_cn = "group_in_slapd_cn"
|
||||||
|
group_in_slapd_uid = "group_in_slapd_uid"
|
||||||
|
group_in_slapd_object_class = "groupOfNames"
|
||||||
|
group_in_slapd = {
|
||||||
|
"dn": group_in_slapd_dn,
|
||||||
|
"attributes": {
|
||||||
|
"cn": group_in_slapd_cn,
|
||||||
|
"uid": group_in_slapd_uid,
|
||||||
|
"objectClass": group_in_slapd_object_class,
|
||||||
|
"member": [user_in_slapd["dn"]],
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
def mock_slapd_connection(password: str) -> Connection:
|
def mock_slapd_connection(password: str) -> Connection:
|
||||||
"""Create mock SLAPD connection"""
|
"""Create mock SLAPD connection"""
|
||||||
@ -96,5 +123,14 @@ def mock_slapd_connection(password: str) -> Connection:
|
|||||||
"objectClass": "posixAccount",
|
"objectClass": "posixAccount",
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
# Known user and group
|
||||||
|
connection.strategy.add_entry(
|
||||||
|
user_in_slapd["dn"],
|
||||||
|
user_in_slapd["attributes"],
|
||||||
|
)
|
||||||
|
connection.strategy.add_entry(
|
||||||
|
group_in_slapd["dn"],
|
||||||
|
group_in_slapd["attributes"],
|
||||||
|
)
|
||||||
connection.bind()
|
connection.bind()
|
||||||
return connection
|
return connection
|
||||||
|
@ -13,14 +13,26 @@ from authentik.events.system_tasks import TaskStatus
|
|||||||
from authentik.lib.generators import generate_id, generate_key
|
from authentik.lib.generators import generate_id, generate_key
|
||||||
from authentik.lib.sync.outgoing.exceptions import StopSync
|
from authentik.lib.sync.outgoing.exceptions import StopSync
|
||||||
from authentik.lib.utils.reflection import class_to_path
|
from authentik.lib.utils.reflection import class_to_path
|
||||||
from authentik.sources.ldap.models import LDAPSource, LDAPSourcePropertyMapping
|
from authentik.sources.ldap.models import (
|
||||||
|
GroupLDAPSourceConnection,
|
||||||
|
LDAPSource,
|
||||||
|
LDAPSourcePropertyMapping,
|
||||||
|
UserLDAPSourceConnection,
|
||||||
|
)
|
||||||
|
from authentik.sources.ldap.sync.forward_delete_users import DELETE_CHUNK_SIZE
|
||||||
from authentik.sources.ldap.sync.groups import GroupLDAPSynchronizer
|
from authentik.sources.ldap.sync.groups import GroupLDAPSynchronizer
|
||||||
from authentik.sources.ldap.sync.membership import MembershipLDAPSynchronizer
|
from authentik.sources.ldap.sync.membership import MembershipLDAPSynchronizer
|
||||||
from authentik.sources.ldap.sync.users import UserLDAPSynchronizer
|
from authentik.sources.ldap.sync.users import UserLDAPSynchronizer
|
||||||
from authentik.sources.ldap.tasks import ldap_sync, ldap_sync_all
|
from authentik.sources.ldap.tasks import ldap_sync, ldap_sync_all
|
||||||
from authentik.sources.ldap.tests.mock_ad import mock_ad_connection
|
from authentik.sources.ldap.tests.mock_ad import mock_ad_connection
|
||||||
from authentik.sources.ldap.tests.mock_freeipa import mock_freeipa_connection
|
from authentik.sources.ldap.tests.mock_freeipa import mock_freeipa_connection
|
||||||
from authentik.sources.ldap.tests.mock_slapd import mock_slapd_connection
|
from authentik.sources.ldap.tests.mock_slapd import (
|
||||||
|
group_in_slapd_cn,
|
||||||
|
group_in_slapd_uid,
|
||||||
|
mock_slapd_connection,
|
||||||
|
user_in_slapd_cn,
|
||||||
|
user_in_slapd_uid,
|
||||||
|
)
|
||||||
|
|
||||||
LDAP_PASSWORD = generate_key()
|
LDAP_PASSWORD = generate_key()
|
||||||
|
|
||||||
@ -257,12 +269,56 @@ class LDAPSyncTests(TestCase):
|
|||||||
self.source.group_membership_field = "memberUid"
|
self.source.group_membership_field = "memberUid"
|
||||||
self.source.user_object_filter = "(objectClass=posixAccount)"
|
self.source.user_object_filter = "(objectClass=posixAccount)"
|
||||||
self.source.group_object_filter = "(objectClass=posixGroup)"
|
self.source.group_object_filter = "(objectClass=posixGroup)"
|
||||||
|
self.source.user_membership_attribute = "uid"
|
||||||
self.source.user_property_mappings.set(
|
self.source.user_property_mappings.set(
|
||||||
|
[
|
||||||
|
*LDAPSourcePropertyMapping.objects.filter(
|
||||||
|
Q(managed__startswith="goauthentik.io/sources/ldap/default")
|
||||||
|
| Q(managed__startswith="goauthentik.io/sources/ldap/openldap")
|
||||||
|
).all(),
|
||||||
|
LDAPSourcePropertyMapping.objects.create(
|
||||||
|
name="name",
|
||||||
|
expression='return {"attributes": {"uid": list_flatten(ldap.get("uid"))}}',
|
||||||
|
),
|
||||||
|
]
|
||||||
|
)
|
||||||
|
self.source.group_property_mappings.set(
|
||||||
LDAPSourcePropertyMapping.objects.filter(
|
LDAPSourcePropertyMapping.objects.filter(
|
||||||
Q(managed__startswith="goauthentik.io/sources/ldap/default")
|
managed="goauthentik.io/sources/ldap/openldap-cn"
|
||||||
| Q(managed__startswith="goauthentik.io/sources/ldap/openldap")
|
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
self.source.save()
|
||||||
|
user_sync = UserLDAPSynchronizer(self.source)
|
||||||
|
user_sync.sync_full()
|
||||||
|
group_sync = GroupLDAPSynchronizer(self.source)
|
||||||
|
group_sync.sync_full()
|
||||||
|
membership_sync = MembershipLDAPSynchronizer(self.source)
|
||||||
|
membership_sync.sync_full()
|
||||||
|
# Test if membership mapping based on memberUid works.
|
||||||
|
posix_group = Group.objects.filter(name="group-posix").first()
|
||||||
|
self.assertTrue(posix_group.users.filter(name="user-posix").exists())
|
||||||
|
|
||||||
|
def test_sync_groups_openldap_posix_group_nonstandard_membership_attribute(self):
|
||||||
|
"""Test posix group sync"""
|
||||||
|
self.source.object_uniqueness_field = "cn"
|
||||||
|
self.source.group_membership_field = "memberUid"
|
||||||
|
self.source.user_object_filter = "(objectClass=posixAccount)"
|
||||||
|
self.source.group_object_filter = "(objectClass=posixGroup)"
|
||||||
|
self.source.user_membership_attribute = "cn"
|
||||||
|
self.source.user_property_mappings.set(
|
||||||
|
[
|
||||||
|
*LDAPSourcePropertyMapping.objects.filter(
|
||||||
|
Q(managed__startswith="goauthentik.io/sources/ldap/default")
|
||||||
|
| Q(managed__startswith="goauthentik.io/sources/ldap/openldap")
|
||||||
|
).all(),
|
||||||
|
LDAPSourcePropertyMapping.objects.create(
|
||||||
|
name="name",
|
||||||
|
expression='return {"attributes": {"cn": list_flatten(ldap.get("cn"))}}',
|
||||||
|
),
|
||||||
|
]
|
||||||
|
)
|
||||||
self.source.group_property_mappings.set(
|
self.source.group_property_mappings.set(
|
||||||
LDAPSourcePropertyMapping.objects.filter(
|
LDAPSourcePropertyMapping.objects.filter(
|
||||||
managed="goauthentik.io/sources/ldap/openldap-cn"
|
managed="goauthentik.io/sources/ldap/openldap-cn"
|
||||||
@ -308,3 +364,160 @@ class LDAPSyncTests(TestCase):
|
|||||||
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
ldap_sync_all.delay().get()
|
ldap_sync_all.delay().get()
|
||||||
|
|
||||||
|
def test_user_deletion(self):
|
||||||
|
"""Test user deletion"""
|
||||||
|
user = User.objects.create_user(username="not-in-the-source")
|
||||||
|
UserLDAPSourceConnection.objects.create(
|
||||||
|
user=user, source=self.source, identifier="not-in-the-source"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertFalse(User.objects.filter(username="not-in-the-source").exists())
|
||||||
|
|
||||||
|
def test_user_deletion_still_in_source(self):
|
||||||
|
"""Test that user is not deleted if it's still in the source"""
|
||||||
|
username = user_in_slapd_cn
|
||||||
|
identifier = user_in_slapd_uid
|
||||||
|
user = User.objects.create_user(username=username)
|
||||||
|
UserLDAPSourceConnection.objects.create(
|
||||||
|
user=user, source=self.source, identifier=identifier
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertTrue(User.objects.filter(username=username).exists())
|
||||||
|
|
||||||
|
def test_user_deletion_no_sync(self):
|
||||||
|
"""Test that user is not deleted if sync_users is False"""
|
||||||
|
user = User.objects.create_user(username="not-in-the-source")
|
||||||
|
UserLDAPSourceConnection.objects.create(
|
||||||
|
user=user, source=self.source, identifier="not-in-the-source"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.sync_users = False
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertTrue(User.objects.filter(username="not-in-the-source").exists())
|
||||||
|
|
||||||
|
def test_user_deletion_no_delete(self):
|
||||||
|
"""Test that user is not deleted if delete_not_found_objects is False"""
|
||||||
|
user = User.objects.create_user(username="not-in-the-source")
|
||||||
|
UserLDAPSourceConnection.objects.create(
|
||||||
|
user=user, source=self.source, identifier="not-in-the-source"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertTrue(User.objects.filter(username="not-in-the-source").exists())
|
||||||
|
|
||||||
|
def test_group_deletion(self):
|
||||||
|
"""Test group deletion"""
|
||||||
|
group = Group.objects.create(name="not-in-the-source")
|
||||||
|
GroupLDAPSourceConnection.objects.create(
|
||||||
|
group=group, source=self.source, identifier="not-in-the-source"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertFalse(Group.objects.filter(name="not-in-the-source").exists())
|
||||||
|
|
||||||
|
def test_group_deletion_still_in_source(self):
|
||||||
|
"""Test that group is not deleted if it's still in the source"""
|
||||||
|
groupname = group_in_slapd_cn
|
||||||
|
identifier = group_in_slapd_uid
|
||||||
|
group = Group.objects.create(name=groupname)
|
||||||
|
GroupLDAPSourceConnection.objects.create(
|
||||||
|
group=group, source=self.source, identifier=identifier
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertTrue(Group.objects.filter(name=groupname).exists())
|
||||||
|
|
||||||
|
def test_group_deletion_no_sync(self):
|
||||||
|
"""Test that group is not deleted if sync_groups is False"""
|
||||||
|
group = Group.objects.create(name="not-in-the-source")
|
||||||
|
GroupLDAPSourceConnection.objects.create(
|
||||||
|
group=group, source=self.source, identifier="not-in-the-source"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.sync_groups = False
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertTrue(Group.objects.filter(name="not-in-the-source").exists())
|
||||||
|
|
||||||
|
def test_group_deletion_no_delete(self):
|
||||||
|
"""Test that group is not deleted if delete_not_found_objects is False"""
|
||||||
|
group = Group.objects.create(name="not-in-the-source")
|
||||||
|
GroupLDAPSourceConnection.objects.create(
|
||||||
|
group=group, source=self.source, identifier="not-in-the-source"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
self.assertTrue(Group.objects.filter(name="not-in-the-source").exists())
|
||||||
|
|
||||||
|
def test_batch_deletion(self):
|
||||||
|
"""Test batch deletion"""
|
||||||
|
BATCH_SIZE = DELETE_CHUNK_SIZE + 1
|
||||||
|
for i in range(BATCH_SIZE):
|
||||||
|
user = User.objects.create_user(username=f"not-in-the-source-{i}")
|
||||||
|
group = Group.objects.create(name=f"not-in-the-source-{i}")
|
||||||
|
group.users.add(user)
|
||||||
|
UserLDAPSourceConnection.objects.create(
|
||||||
|
user=user, source=self.source, identifier=f"not-in-the-source-{i}-user"
|
||||||
|
)
|
||||||
|
GroupLDAPSourceConnection.objects.create(
|
||||||
|
group=group, source=self.source, identifier=f"not-in-the-source-{i}-group"
|
||||||
|
)
|
||||||
|
self.source.object_uniqueness_field = "uid"
|
||||||
|
self.source.group_object_filter = "(objectClass=groupOfNames)"
|
||||||
|
self.source.delete_not_found_objects = True
|
||||||
|
self.source.save()
|
||||||
|
|
||||||
|
connection = MagicMock(return_value=mock_slapd_connection(LDAP_PASSWORD))
|
||||||
|
with patch("authentik.sources.ldap.models.LDAPSource.connection", connection):
|
||||||
|
ldap_sync_all.delay().get()
|
||||||
|
|
||||||
|
self.assertFalse(User.objects.filter(username__startswith="not-in-the-source").exists())
|
||||||
|
self.assertFalse(Group.objects.filter(name__startswith="not-in-the-source").exists())
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik plex config"""
|
"""authentik plex config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikSourcePlexConfig(AppConfig):
|
class AuthentikSourcePlexConfig(ManagedAppConfig):
|
||||||
"""authentik source plex config"""
|
"""authentik source plex config"""
|
||||||
|
|
||||||
name = "authentik.sources.plex"
|
name = "authentik.sources.plex"
|
||||||
label = "authentik_sources_plex"
|
label = "authentik_sources_plex"
|
||||||
verbose_name = "authentik Sources.Plex"
|
verbose_name = "authentik Sources.Plex"
|
||||||
|
default = True
|
||||||
|
@ -9,6 +9,7 @@ from django.http.response import HttpResponseBadRequest
|
|||||||
from django.shortcuts import get_object_or_404, redirect
|
from django.shortcuts import get_object_or_404, redirect
|
||||||
from django.utils.decorators import method_decorator
|
from django.utils.decorators import method_decorator
|
||||||
from django.utils.http import urlencode
|
from django.utils.http import urlencode
|
||||||
|
from django.utils.translation import gettext as _
|
||||||
from django.views import View
|
from django.views import View
|
||||||
from django.views.decorators.csrf import csrf_exempt
|
from django.views.decorators.csrf import csrf_exempt
|
||||||
from structlog.stdlib import get_logger
|
from structlog.stdlib import get_logger
|
||||||
@ -128,7 +129,9 @@ class InitiateView(View):
|
|||||||
# otherwise we default to POST_AUTO, with direct redirect
|
# otherwise we default to POST_AUTO, with direct redirect
|
||||||
if source.binding_type == SAMLBindingTypes.POST:
|
if source.binding_type == SAMLBindingTypes.POST:
|
||||||
injected_stages.append(in_memory_stage(ConsentStageView))
|
injected_stages.append(in_memory_stage(ConsentStageView))
|
||||||
plan_kwargs[PLAN_CONTEXT_CONSENT_HEADER] = f"Continue to {source.name}"
|
plan_kwargs[PLAN_CONTEXT_CONSENT_HEADER] = _(
|
||||||
|
"Continue to {source_name}".format(source_name=source.name)
|
||||||
|
)
|
||||||
injected_stages.append(in_memory_stage(AutosubmitStageView))
|
injected_stages.append(in_memory_stage(AutosubmitStageView))
|
||||||
return self.handle_login_flow(
|
return self.handle_login_flow(
|
||||||
source,
|
source,
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""Authenticator"""
|
"""Authenticator"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageAuthenticatorConfig(AppConfig):
|
class AuthentikStageAuthenticatorConfig(ManagedAppConfig):
|
||||||
"""Authenticator App config"""
|
"""Authenticator App config"""
|
||||||
|
|
||||||
name = "authentik.stages.authenticator"
|
name = "authentik.stages.authenticator"
|
||||||
label = "authentik_stages_authenticator"
|
label = "authentik_stages_authenticator"
|
||||||
verbose_name = "authentik Stages.Authenticator"
|
verbose_name = "authentik Stages.Authenticator"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""SMS"""
|
"""SMS"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageAuthenticatorSMSConfig(AppConfig):
|
class AuthentikStageAuthenticatorSMSConfig(ManagedAppConfig):
|
||||||
"""SMS App config"""
|
"""SMS App config"""
|
||||||
|
|
||||||
name = "authentik.stages.authenticator_sms"
|
name = "authentik.stages.authenticator_sms"
|
||||||
label = "authentik_stages_authenticator_sms"
|
label = "authentik_stages_authenticator_sms"
|
||||||
verbose_name = "authentik Stages.Authenticator.SMS"
|
verbose_name = "authentik Stages.Authenticator.SMS"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""TOTP"""
|
"""TOTP"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageAuthenticatorTOTPConfig(AppConfig):
|
class AuthentikStageAuthenticatorTOTPConfig(ManagedAppConfig):
|
||||||
"""TOTP App config"""
|
"""TOTP App config"""
|
||||||
|
|
||||||
name = "authentik.stages.authenticator_totp"
|
name = "authentik.stages.authenticator_totp"
|
||||||
label = "authentik_stages_authenticator_totp"
|
label = "authentik_stages_authenticator_totp"
|
||||||
verbose_name = "authentik Stages.Authenticator.TOTP"
|
verbose_name = "authentik Stages.Authenticator.TOTP"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""Authenticator Validation Stage"""
|
"""Authenticator Validation Stage"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageAuthenticatorValidateConfig(AppConfig):
|
class AuthentikStageAuthenticatorValidateConfig(ManagedAppConfig):
|
||||||
"""Authenticator Validation Stage"""
|
"""Authenticator Validation Stage"""
|
||||||
|
|
||||||
name = "authentik.stages.authenticator_validate"
|
name = "authentik.stages.authenticator_validate"
|
||||||
label = "authentik_stages_authenticator_validate"
|
label = "authentik_stages_authenticator_validate"
|
||||||
verbose_name = "authentik Stages.Authenticator.Validate"
|
verbose_name = "authentik Stages.Authenticator.Validate"
|
||||||
|
default = True
|
||||||
|
@ -168,13 +168,10 @@ class AuthenticatorValidateStageDuoTests(FlowTestCase):
|
|||||||
user__pk=self.user.pk,
|
user__pk=self.user.pk,
|
||||||
).first()
|
).first()
|
||||||
self.assertIsNotNone(event)
|
self.assertIsNotNone(event)
|
||||||
context = dict(event.context)
|
|
||||||
# The auth_method field is being obfuscated as it's marked as sensitive in Django 5.2
|
|
||||||
auth_method = context.pop("auth_method")
|
|
||||||
self.assertIn(auth_method, ["auth_mfa", "********************"])
|
|
||||||
self.assertEqual(
|
self.assertEqual(
|
||||||
context,
|
event.context,
|
||||||
{
|
{
|
||||||
|
"auth_method": "auth_mfa",
|
||||||
"auth_method_args": {
|
"auth_method_args": {
|
||||||
"mfa_devices": [
|
"mfa_devices": [
|
||||||
{
|
{
|
||||||
|
@ -151,9 +151,7 @@ class AuthenticatorValidateStageWebAuthnTests(FlowTestCase):
|
|||||||
webauthn_user_verification=UserVerification.PREFERRED,
|
webauthn_user_verification=UserVerification.PREFERRED,
|
||||||
)
|
)
|
||||||
stage.webauthn_allowed_device_types.set(
|
stage.webauthn_allowed_device_types.set(
|
||||||
WebAuthnDeviceType.objects.filter(
|
WebAuthnDeviceType.objects.filter(description="YubiKey 5 Series")
|
||||||
description="Android Authenticator with SafetyNet Attestation"
|
|
||||||
)
|
|
||||||
)
|
)
|
||||||
session = self.client.session
|
session = self.client.session
|
||||||
plan = FlowPlan(flow_pk=flow.pk.hex)
|
plan = FlowPlan(flow_pk=flow.pk.hex)
|
||||||
@ -339,9 +337,7 @@ class AuthenticatorValidateStageWebAuthnTests(FlowTestCase):
|
|||||||
device_classes=[DeviceClasses.WEBAUTHN],
|
device_classes=[DeviceClasses.WEBAUTHN],
|
||||||
)
|
)
|
||||||
stage.webauthn_allowed_device_types.set(
|
stage.webauthn_allowed_device_types.set(
|
||||||
WebAuthnDeviceType.objects.filter(
|
WebAuthnDeviceType.objects.filter(description="YubiKey 5 Series")
|
||||||
description="Android Authenticator with SafetyNet Attestation"
|
|
||||||
)
|
|
||||||
)
|
)
|
||||||
session = self.client.session
|
session = self.client.session
|
||||||
plan = FlowPlan(flow_pk=flow.pk.hex)
|
plan = FlowPlan(flow_pk=flow.pk.hex)
|
||||||
|
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
@ -141,9 +141,7 @@ class TestAuthenticatorWebAuthnStage(FlowTestCase):
|
|||||||
"""Test registration with restricted devices (fail)"""
|
"""Test registration with restricted devices (fail)"""
|
||||||
webauthn_mds_import.delay(force=True).get()
|
webauthn_mds_import.delay(force=True).get()
|
||||||
self.stage.device_type_restrictions.set(
|
self.stage.device_type_restrictions.set(
|
||||||
WebAuthnDeviceType.objects.filter(
|
WebAuthnDeviceType.objects.filter(description="YubiKey 5 Series")
|
||||||
description="Android Authenticator with SafetyNet Attestation"
|
|
||||||
)
|
|
||||||
)
|
)
|
||||||
|
|
||||||
plan = FlowPlan(flow_pk=self.flow.pk.hex, bindings=[self.binding], markers=[StageMarker()])
|
plan = FlowPlan(flow_pk=self.flow.pk.hex, bindings=[self.binding], markers=[StageMarker()])
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik captcha app"""
|
"""authentik captcha app"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageCaptchaConfig(AppConfig):
|
class AuthentikStageCaptchaConfig(ManagedAppConfig):
|
||||||
"""authentik captcha app"""
|
"""authentik captcha app"""
|
||||||
|
|
||||||
name = "authentik.stages.captcha"
|
name = "authentik.stages.captcha"
|
||||||
label = "authentik_stages_captcha"
|
label = "authentik_stages_captcha"
|
||||||
verbose_name = "authentik Stages.Captcha"
|
verbose_name = "authentik Stages.Captcha"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik consent app"""
|
"""authentik consent app"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageConsentConfig(AppConfig):
|
class AuthentikStageConsentConfig(ManagedAppConfig):
|
||||||
"""authentik consent app"""
|
"""authentik consent app"""
|
||||||
|
|
||||||
name = "authentik.stages.consent"
|
name = "authentik.stages.consent"
|
||||||
label = "authentik_stages_consent"
|
label = "authentik_stages_consent"
|
||||||
verbose_name = "authentik Stages.Consent"
|
verbose_name = "authentik Stages.Consent"
|
||||||
|
default = True
|
||||||
|
@ -4,6 +4,8 @@ from uuid import uuid4
|
|||||||
|
|
||||||
from django.http import HttpRequest, HttpResponse
|
from django.http import HttpRequest, HttpResponse
|
||||||
from django.utils.timezone import now
|
from django.utils.timezone import now
|
||||||
|
from django.utils.translation import gettext as _
|
||||||
|
from rest_framework.exceptions import ValidationError
|
||||||
from rest_framework.fields import CharField
|
from rest_framework.fields import CharField
|
||||||
|
|
||||||
from authentik.core.api.utils import PassiveSerializer
|
from authentik.core.api.utils import PassiveSerializer
|
||||||
@ -47,6 +49,11 @@ class ConsentChallengeResponse(ChallengeResponse):
|
|||||||
component = CharField(default="ak-stage-consent")
|
component = CharField(default="ak-stage-consent")
|
||||||
token = CharField(required=True)
|
token = CharField(required=True)
|
||||||
|
|
||||||
|
def validate_token(self, token: str):
|
||||||
|
if token != self.stage.executor.request.session[SESSION_KEY_CONSENT_TOKEN]:
|
||||||
|
raise ValidationError(_("Invalid consent token, re-showing prompt"))
|
||||||
|
return token
|
||||||
|
|
||||||
|
|
||||||
class ConsentStageView(ChallengeStageView):
|
class ConsentStageView(ChallengeStageView):
|
||||||
"""Simple consent checker."""
|
"""Simple consent checker."""
|
||||||
@ -120,9 +127,6 @@ class ConsentStageView(ChallengeStageView):
|
|||||||
return super().get(request, *args, **kwargs)
|
return super().get(request, *args, **kwargs)
|
||||||
|
|
||||||
def challenge_valid(self, response: ChallengeResponse) -> HttpResponse:
|
def challenge_valid(self, response: ChallengeResponse) -> HttpResponse:
|
||||||
if response.data["token"] != self.request.session[SESSION_KEY_CONSENT_TOKEN]:
|
|
||||||
self.logger.info("Invalid consent token, re-showing prompt")
|
|
||||||
return self.get(self.request)
|
|
||||||
if self.should_always_prompt():
|
if self.should_always_prompt():
|
||||||
return self.executor.stage_ok()
|
return self.executor.stage_ok()
|
||||||
current_stage: ConsentStage = self.executor.current_stage
|
current_stage: ConsentStage = self.executor.current_stage
|
||||||
|
@ -17,6 +17,7 @@ from authentik.flows.views.executor import SESSION_KEY_PLAN
|
|||||||
from authentik.lib.generators import generate_id
|
from authentik.lib.generators import generate_id
|
||||||
from authentik.stages.consent.models import ConsentMode, ConsentStage, UserConsent
|
from authentik.stages.consent.models import ConsentMode, ConsentStage, UserConsent
|
||||||
from authentik.stages.consent.stage import (
|
from authentik.stages.consent.stage import (
|
||||||
|
PLAN_CONTEXT_CONSENT_HEADER,
|
||||||
PLAN_CONTEXT_CONSENT_PERMISSIONS,
|
PLAN_CONTEXT_CONSENT_PERMISSIONS,
|
||||||
SESSION_KEY_CONSENT_TOKEN,
|
SESSION_KEY_CONSENT_TOKEN,
|
||||||
)
|
)
|
||||||
@ -33,6 +34,40 @@ class TestConsentStage(FlowTestCase):
|
|||||||
slug=generate_id(),
|
slug=generate_id(),
|
||||||
)
|
)
|
||||||
|
|
||||||
|
def test_mismatched_token(self):
|
||||||
|
"""Test incorrect token"""
|
||||||
|
flow = create_test_flow(FlowDesignation.AUTHENTICATION)
|
||||||
|
stage = ConsentStage.objects.create(name=generate_id(), mode=ConsentMode.ALWAYS_REQUIRE)
|
||||||
|
binding = FlowStageBinding.objects.create(target=flow, stage=stage, order=2)
|
||||||
|
|
||||||
|
plan = FlowPlan(flow_pk=flow.pk.hex, bindings=[binding], markers=[StageMarker()])
|
||||||
|
session = self.client.session
|
||||||
|
session[SESSION_KEY_PLAN] = plan
|
||||||
|
session.save()
|
||||||
|
response = self.client.get(
|
||||||
|
reverse("authentik_api:flow-executor", kwargs={"flow_slug": flow.slug}),
|
||||||
|
)
|
||||||
|
self.assertEqual(response.status_code, 200)
|
||||||
|
|
||||||
|
session = self.client.session
|
||||||
|
response = self.client.post(
|
||||||
|
reverse("authentik_api:flow-executor", kwargs={"flow_slug": flow.slug}),
|
||||||
|
{
|
||||||
|
"token": generate_id(),
|
||||||
|
},
|
||||||
|
)
|
||||||
|
|
||||||
|
self.assertEqual(response.status_code, 200)
|
||||||
|
self.assertStageResponse(
|
||||||
|
response,
|
||||||
|
flow,
|
||||||
|
component="ak-stage-consent",
|
||||||
|
response_errors={
|
||||||
|
"token": [{"string": "Invalid consent token, re-showing prompt", "code": "invalid"}]
|
||||||
|
},
|
||||||
|
)
|
||||||
|
self.assertFalse(UserConsent.objects.filter(user=self.user).exists())
|
||||||
|
|
||||||
def test_always_required(self):
|
def test_always_required(self):
|
||||||
"""Test always required consent"""
|
"""Test always required consent"""
|
||||||
flow = create_test_flow(FlowDesignation.AUTHENTICATION)
|
flow = create_test_flow(FlowDesignation.AUTHENTICATION)
|
||||||
@ -158,6 +193,7 @@ class TestConsentStage(FlowTestCase):
|
|||||||
context={
|
context={
|
||||||
PLAN_CONTEXT_APPLICATION: self.application,
|
PLAN_CONTEXT_APPLICATION: self.application,
|
||||||
PLAN_CONTEXT_CONSENT_PERMISSIONS: [PermissionDict(id="foo", name="foo-desc")],
|
PLAN_CONTEXT_CONSENT_PERMISSIONS: [PermissionDict(id="foo", name="foo-desc")],
|
||||||
|
PLAN_CONTEXT_CONSENT_HEADER: "test header",
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
session = self.client.session
|
session = self.client.session
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik deny stage app config"""
|
"""authentik deny stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageDenyConfig(AppConfig):
|
class AuthentikStageDenyConfig(ManagedAppConfig):
|
||||||
"""authentik deny stage config"""
|
"""authentik deny stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.deny"
|
name = "authentik.stages.deny"
|
||||||
label = "authentik_stages_deny"
|
label = "authentik_stages_deny"
|
||||||
verbose_name = "authentik Stages.Deny"
|
verbose_name = "authentik Stages.Deny"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik dummy stage config"""
|
"""authentik dummy stage config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageDummyConfig(AppConfig):
|
class AuthentikStageDummyConfig(ManagedAppConfig):
|
||||||
"""authentik dummy stage config"""
|
"""authentik dummy stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.dummy"
|
name = "authentik.stages.dummy"
|
||||||
label = "authentik_stages_dummy"
|
label = "authentik_stages_dummy"
|
||||||
verbose_name = "authentik Stages.Dummy"
|
verbose_name = "authentik Stages.Dummy"
|
||||||
|
default = True
|
||||||
|
38
authentik/stages/email/flow.py
Normal file
38
authentik/stages/email/flow.py
Normal file
@ -0,0 +1,38 @@
|
|||||||
|
from base64 import b64encode
|
||||||
|
from copy import deepcopy
|
||||||
|
from pickle import dumps # nosec
|
||||||
|
|
||||||
|
from django.utils.translation import gettext as _
|
||||||
|
|
||||||
|
from authentik.flows.models import FlowToken, in_memory_stage
|
||||||
|
from authentik.flows.planner import PLAN_CONTEXT_IS_RESTORED, FlowPlan
|
||||||
|
from authentik.stages.consent.stage import PLAN_CONTEXT_CONSENT_HEADER, ConsentStageView
|
||||||
|
|
||||||
|
|
||||||
|
def pickle_flow_token_for_email(plan: FlowPlan):
|
||||||
|
"""Insert a consent stage into the flow plan and pickle it for a FlowToken,
|
||||||
|
to be sent via Email. This is to prevent automated email scanners, which sometimes
|
||||||
|
open links in emails in a full browser from breaking the link."""
|
||||||
|
plan_copy = deepcopy(plan)
|
||||||
|
plan_copy.insert_stage(in_memory_stage(EmailTokenRevocationConsentStageView), index=0)
|
||||||
|
plan_copy.context[PLAN_CONTEXT_CONSENT_HEADER] = _("Continue to confirm this email address.")
|
||||||
|
data = dumps(plan_copy)
|
||||||
|
return b64encode(data).decode()
|
||||||
|
|
||||||
|
|
||||||
|
class EmailTokenRevocationConsentStageView(ConsentStageView):
|
||||||
|
|
||||||
|
def get(self, request, *args, **kwargs):
|
||||||
|
token: FlowToken = self.executor.plan.context[PLAN_CONTEXT_IS_RESTORED]
|
||||||
|
try:
|
||||||
|
token.refresh_from_db()
|
||||||
|
except FlowToken.DoesNotExist:
|
||||||
|
return self.executor.stage_invalid(
|
||||||
|
_("Link was already used, please request a new link.")
|
||||||
|
)
|
||||||
|
return super().get(request, *args, **kwargs)
|
||||||
|
|
||||||
|
def challenge_valid(self, response):
|
||||||
|
token: FlowToken = self.executor.plan.context[PLAN_CONTEXT_IS_RESTORED]
|
||||||
|
token.delete()
|
||||||
|
return super().challenge_valid(response)
|
@ -23,6 +23,7 @@ from authentik.flows.stage import ChallengeStageView
|
|||||||
from authentik.flows.views.executor import QS_KEY_TOKEN, QS_QUERY
|
from authentik.flows.views.executor import QS_KEY_TOKEN, QS_QUERY
|
||||||
from authentik.lib.utils.errors import exception_to_string
|
from authentik.lib.utils.errors import exception_to_string
|
||||||
from authentik.lib.utils.time import timedelta_from_string
|
from authentik.lib.utils.time import timedelta_from_string
|
||||||
|
from authentik.stages.email.flow import pickle_flow_token_for_email
|
||||||
from authentik.stages.email.models import EmailStage
|
from authentik.stages.email.models import EmailStage
|
||||||
from authentik.stages.email.tasks import send_mails
|
from authentik.stages.email.tasks import send_mails
|
||||||
from authentik.stages.email.utils import TemplateEmailMessage
|
from authentik.stages.email.utils import TemplateEmailMessage
|
||||||
@ -86,7 +87,8 @@ class EmailStageView(ChallengeStageView):
|
|||||||
user=pending_user,
|
user=pending_user,
|
||||||
identifier=identifier,
|
identifier=identifier,
|
||||||
flow=self.executor.flow,
|
flow=self.executor.flow,
|
||||||
_plan=FlowToken.pickle(self.executor.plan),
|
_plan=pickle_flow_token_for_email(self.executor.plan),
|
||||||
|
revoke_on_execution=False,
|
||||||
)
|
)
|
||||||
token = tokens.first()
|
token = tokens.first()
|
||||||
# Check if token is expired and rotate key if so
|
# Check if token is expired and rotate key if so
|
||||||
|
@ -174,5 +174,5 @@ class TestEmailStageSending(FlowTestCase):
|
|||||||
response = self.client.post(url)
|
response = self.client.post(url)
|
||||||
response = self.client.post(url)
|
response = self.client.post(url)
|
||||||
self.assertEqual(response.status_code, 200)
|
self.assertEqual(response.status_code, 200)
|
||||||
self.assertTrue(len(mail.outbox) >= 1)
|
self.assertGreaterEqual(len(mail.outbox), 1)
|
||||||
self.assertEqual(mail.outbox[0].subject, "authentik")
|
self.assertEqual(mail.outbox[0].subject, "authentik")
|
||||||
|
@ -17,6 +17,7 @@ from authentik.flows.tests import FlowTestCase
|
|||||||
from authentik.flows.views.executor import QS_KEY_TOKEN, SESSION_KEY_PLAN, FlowExecutorView
|
from authentik.flows.views.executor import QS_KEY_TOKEN, SESSION_KEY_PLAN, FlowExecutorView
|
||||||
from authentik.lib.config import CONFIG
|
from authentik.lib.config import CONFIG
|
||||||
from authentik.lib.generators import generate_id
|
from authentik.lib.generators import generate_id
|
||||||
|
from authentik.stages.consent.stage import SESSION_KEY_CONSENT_TOKEN
|
||||||
from authentik.stages.email.models import EmailStage
|
from authentik.stages.email.models import EmailStage
|
||||||
from authentik.stages.email.stage import PLAN_CONTEXT_EMAIL_OVERRIDE, EmailStageView
|
from authentik.stages.email.stage import PLAN_CONTEXT_EMAIL_OVERRIDE, EmailStageView
|
||||||
|
|
||||||
@ -160,6 +161,17 @@ class TestEmailStage(FlowTestCase):
|
|||||||
kwargs={"flow_slug": self.flow.slug},
|
kwargs={"flow_slug": self.flow.slug},
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
self.assertStageResponse(response, self.flow, component="ak-stage-consent")
|
||||||
|
response = self.client.post(
|
||||||
|
reverse(
|
||||||
|
"authentik_api:flow-executor",
|
||||||
|
kwargs={"flow_slug": self.flow.slug},
|
||||||
|
),
|
||||||
|
data={
|
||||||
|
"token": self.client.session[SESSION_KEY_CONSENT_TOKEN],
|
||||||
|
},
|
||||||
|
follow=True,
|
||||||
|
)
|
||||||
|
|
||||||
self.assertEqual(response.status_code, 200)
|
self.assertEqual(response.status_code, 200)
|
||||||
self.assertStageRedirects(response, reverse("authentik_core:root-redirect"))
|
self.assertStageRedirects(response, reverse("authentik_core:root-redirect"))
|
||||||
@ -182,6 +194,7 @@ class TestEmailStage(FlowTestCase):
|
|||||||
# Set flow token user to a different user
|
# Set flow token user to a different user
|
||||||
token: FlowToken = FlowToken.objects.get(user=self.user)
|
token: FlowToken = FlowToken.objects.get(user=self.user)
|
||||||
token.user = create_test_admin_user()
|
token.user = create_test_admin_user()
|
||||||
|
token.revoke_on_execution = True
|
||||||
token.save()
|
token.save()
|
||||||
|
|
||||||
with patch("authentik.flows.views.executor.FlowExecutorView.cancel", MagicMock()):
|
with patch("authentik.flows.views.executor.FlowExecutorView.cancel", MagicMock()):
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik identification stage app config"""
|
"""authentik identification stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageIdentificationConfig(AppConfig):
|
class AuthentikStageIdentificationConfig(ManagedAppConfig):
|
||||||
"""authentik identification stage config"""
|
"""authentik identification stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.identification"
|
name = "authentik.stages.identification"
|
||||||
label = "authentik_stages_identification"
|
label = "authentik_stages_identification"
|
||||||
verbose_name = "authentik Stages.Identification"
|
verbose_name = "authentik Stages.Identification"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik invitation stage app config"""
|
"""authentik invitation stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageInvitationConfig(AppConfig):
|
class AuthentikStageInvitationConfig(ManagedAppConfig):
|
||||||
"""authentik invitation stage config"""
|
"""authentik invitation stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.invitation"
|
name = "authentik.stages.invitation"
|
||||||
label = "authentik_stages_invitation"
|
label = "authentik_stages_invitation"
|
||||||
verbose_name = "authentik Stages.Invitation"
|
verbose_name = "authentik Stages.Invitation"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik core app config"""
|
"""authentik core app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStagePasswordConfig(AppConfig):
|
class AuthentikStagePasswordConfig(ManagedAppConfig):
|
||||||
"""authentik password stage config"""
|
"""authentik password stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.password"
|
name = "authentik.stages.password"
|
||||||
label = "authentik_stages_password"
|
label = "authentik_stages_password"
|
||||||
verbose_name = "authentik Stages.Password"
|
verbose_name = "authentik Stages.Password"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik prompt stage app config"""
|
"""authentik prompt stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStagePromptConfig(AppConfig):
|
class AuthentikStagePromptConfig(ManagedAppConfig):
|
||||||
"""authentik prompt stage config"""
|
"""authentik prompt stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.prompt"
|
name = "authentik.stages.prompt"
|
||||||
label = "authentik_stages_prompt"
|
label = "authentik_stages_prompt"
|
||||||
verbose_name = "authentik Stages.Prompt"
|
verbose_name = "authentik Stages.Prompt"
|
||||||
|
default = True
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
"""Prompt Stage Logic"""
|
"""Prompt Stage Logic"""
|
||||||
|
|
||||||
from collections.abc import Callable, Iterator
|
from collections.abc import Callable
|
||||||
from email.policy import Policy
|
from email.policy import Policy
|
||||||
from types import MethodType
|
from types import MethodType
|
||||||
from typing import Any
|
from typing import Any
|
||||||
@ -190,7 +190,7 @@ class ListPolicyEngine(PolicyEngine):
|
|||||||
self.__list = policies
|
self.__list = policies
|
||||||
self.use_cache = False
|
self.use_cache = False
|
||||||
|
|
||||||
def iterate_bindings(self) -> Iterator[PolicyBinding]:
|
def bindings(self):
|
||||||
for policy in self.__list:
|
for policy in self.__list:
|
||||||
yield PolicyBinding(
|
yield PolicyBinding(
|
||||||
policy=policy,
|
policy=policy,
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik redirect app"""
|
"""authentik redirect app"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageRedirectConfig(AppConfig):
|
class AuthentikStageRedirectConfig(ManagedAppConfig):
|
||||||
"""authentik redirect app"""
|
"""authentik redirect app"""
|
||||||
|
|
||||||
name = "authentik.stages.redirect"
|
name = "authentik.stages.redirect"
|
||||||
label = "authentik_stages_redirect"
|
label = "authentik_stages_redirect"
|
||||||
verbose_name = "authentik Stages.Redirect"
|
verbose_name = "authentik Stages.Redirect"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik delete stage app config"""
|
"""authentik delete stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageUserDeleteConfig(AppConfig):
|
class AuthentikStageUserDeleteConfig(ManagedAppConfig):
|
||||||
"""authentik delete stage config"""
|
"""authentik delete stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.user_delete"
|
name = "authentik.stages.user_delete"
|
||||||
label = "authentik_stages_user_delete"
|
label = "authentik_stages_user_delete"
|
||||||
verbose_name = "authentik Stages.User Delete"
|
verbose_name = "authentik Stages.User Delete"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik login stage app config"""
|
"""authentik login stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageUserLoginConfig(AppConfig):
|
class AuthentikStageUserLoginConfig(ManagedAppConfig):
|
||||||
"""authentik login stage config"""
|
"""authentik login stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.user_login"
|
name = "authentik.stages.user_login"
|
||||||
label = "authentik_stages_user_login"
|
label = "authentik_stages_user_login"
|
||||||
verbose_name = "authentik Stages.User Login"
|
verbose_name = "authentik Stages.User Login"
|
||||||
|
default = True
|
||||||
|
@ -11,7 +11,7 @@ from rest_framework.fields import BooleanField, CharField
|
|||||||
from authentik.core.models import Session, User
|
from authentik.core.models import Session, User
|
||||||
from authentik.events.middleware import audit_ignore
|
from authentik.events.middleware import audit_ignore
|
||||||
from authentik.flows.challenge import ChallengeResponse, WithUserInfoChallenge
|
from authentik.flows.challenge import ChallengeResponse, WithUserInfoChallenge
|
||||||
from authentik.flows.planner import PLAN_CONTEXT_PENDING_USER, PLAN_CONTEXT_SOURCE
|
from authentik.flows.planner import PLAN_CONTEXT_PENDING_USER
|
||||||
from authentik.flows.stage import ChallengeStageView
|
from authentik.flows.stage import ChallengeStageView
|
||||||
from authentik.lib.utils.time import timedelta_from_string
|
from authentik.lib.utils.time import timedelta_from_string
|
||||||
from authentik.root.middleware import ClientIPMiddleware
|
from authentik.root.middleware import ClientIPMiddleware
|
||||||
@ -108,10 +108,6 @@ class UserLoginStageView(ChallengeStageView):
|
|||||||
flow_slug=self.executor.flow.slug,
|
flow_slug=self.executor.flow.slug,
|
||||||
session_duration=delta,
|
session_duration=delta,
|
||||||
)
|
)
|
||||||
# Only show success message if we don't have a source in the flow
|
|
||||||
# as sources show their own success messages
|
|
||||||
if not self.executor.plan.context.get(PLAN_CONTEXT_SOURCE, None):
|
|
||||||
messages.success(self.request, _("Successfully logged in!"))
|
|
||||||
if self.executor.current_stage.terminate_other_sessions:
|
if self.executor.current_stage.terminate_other_sessions:
|
||||||
Session.objects.filter(
|
Session.objects.filter(
|
||||||
authenticatedsession__user=user,
|
authenticatedsession__user=user,
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik logout stage app config"""
|
"""authentik logout stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageUserLogoutConfig(AppConfig):
|
class AuthentikStageUserLogoutConfig(ManagedAppConfig):
|
||||||
"""authentik logout stage config"""
|
"""authentik logout stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.user_logout"
|
name = "authentik.stages.user_logout"
|
||||||
label = "authentik_stages_user_logout"
|
label = "authentik_stages_user_logout"
|
||||||
verbose_name = "authentik Stages.User Logout"
|
verbose_name = "authentik Stages.User Logout"
|
||||||
|
default = True
|
||||||
|
@ -1,11 +1,12 @@
|
|||||||
"""authentik write stage app config"""
|
"""authentik write stage app config"""
|
||||||
|
|
||||||
from django.apps import AppConfig
|
from authentik.blueprints.apps import ManagedAppConfig
|
||||||
|
|
||||||
|
|
||||||
class AuthentikStageUserWriteConfig(AppConfig):
|
class AuthentikStageUserWriteConfig(ManagedAppConfig):
|
||||||
"""authentik write stage config"""
|
"""authentik write stage config"""
|
||||||
|
|
||||||
name = "authentik.stages.user_write"
|
name = "authentik.stages.user_write"
|
||||||
label = "authentik_stages_user_write"
|
label = "authentik_stages_user_write"
|
||||||
verbose_name = "authentik Stages.User Write"
|
verbose_name = "authentik Stages.User Write"
|
||||||
|
default = True
|
||||||
|
@ -2,7 +2,7 @@
|
|||||||
"$schema": "http://json-schema.org/draft-07/schema",
|
"$schema": "http://json-schema.org/draft-07/schema",
|
||||||
"$id": "https://goauthentik.io/blueprints/schema.json",
|
"$id": "https://goauthentik.io/blueprints/schema.json",
|
||||||
"type": "object",
|
"type": "object",
|
||||||
"title": "authentik 2025.4.1 Blueprint schema",
|
"title": "authentik 2025.6.0 Blueprint schema",
|
||||||
"required": [
|
"required": [
|
||||||
"version",
|
"version",
|
||||||
"entries"
|
"entries"
|
||||||
@ -8147,6 +8147,12 @@
|
|||||||
"title": "Group membership field",
|
"title": "Group membership field",
|
||||||
"description": "Field which contains members of a group."
|
"description": "Field which contains members of a group."
|
||||||
},
|
},
|
||||||
|
"user_membership_attribute": {
|
||||||
|
"type": "string",
|
||||||
|
"minLength": 1,
|
||||||
|
"title": "User membership attribute",
|
||||||
|
"description": "Attribute which matches the value of `group_membership_field`."
|
||||||
|
},
|
||||||
"object_uniqueness_field": {
|
"object_uniqueness_field": {
|
||||||
"type": "string",
|
"type": "string",
|
||||||
"minLength": 1,
|
"minLength": 1,
|
||||||
@ -8180,6 +8186,11 @@
|
|||||||
"type": "boolean",
|
"type": "boolean",
|
||||||
"title": "Lookup groups from user",
|
"title": "Lookup groups from user",
|
||||||
"description": "Lookup group membership based on a user attribute instead of a group attribute. This allows nested group resolution on systems like FreeIPA and Active Directory"
|
"description": "Lookup group membership based on a user attribute instead of a group attribute. This allows nested group resolution on systems like FreeIPA and Active Directory"
|
||||||
|
},
|
||||||
|
"delete_not_found_objects": {
|
||||||
|
"type": "boolean",
|
||||||
|
"title": "Delete not found objects",
|
||||||
|
"description": "Delete authentik users and groups which were previously supplied by this source, but are now missing from it."
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"required": []
|
"required": []
|
||||||
|
@ -2,17 +2,13 @@ package main
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"fmt"
|
"fmt"
|
||||||
"net/url"
|
|
||||||
"os"
|
"os"
|
||||||
|
|
||||||
log "github.com/sirupsen/logrus"
|
|
||||||
"github.com/spf13/cobra"
|
"github.com/spf13/cobra"
|
||||||
|
|
||||||
"goauthentik.io/internal/common"
|
"goauthentik.io/internal/common"
|
||||||
"goauthentik.io/internal/config"
|
|
||||||
"goauthentik.io/internal/constants"
|
"goauthentik.io/internal/constants"
|
||||||
"goauthentik.io/internal/debug"
|
"goauthentik.io/internal/outpost/ak/entrypoint"
|
||||||
"goauthentik.io/internal/outpost/ak"
|
|
||||||
"goauthentik.io/internal/outpost/ak/healthcheck"
|
"goauthentik.io/internal/outpost/ak/healthcheck"
|
||||||
"goauthentik.io/internal/outpost/ldap"
|
"goauthentik.io/internal/outpost/ldap"
|
||||||
)
|
)
|
||||||
@ -25,65 +21,15 @@ Required environment variables:
|
|||||||
- AUTHENTIK_INSECURE: Skip SSL Certificate verification`
|
- AUTHENTIK_INSECURE: Skip SSL Certificate verification`
|
||||||
|
|
||||||
var rootCmd = &cobra.Command{
|
var rootCmd = &cobra.Command{
|
||||||
Long: helpMessage,
|
Long: helpMessage,
|
||||||
Version: constants.FullVersion(),
|
Version: constants.FullVersion(),
|
||||||
PersistentPreRun: func(cmd *cobra.Command, args []string) {
|
PersistentPreRun: common.PreRun,
|
||||||
log.SetLevel(log.DebugLevel)
|
RunE: func(cmd *cobra.Command, args []string) error {
|
||||||
log.SetFormatter(&log.JSONFormatter{
|
err := entrypoint.OutpostMain("authentik.outpost.ldap", ldap.NewServer)
|
||||||
FieldMap: log.FieldMap{
|
|
||||||
log.FieldKeyMsg: "event",
|
|
||||||
log.FieldKeyTime: "timestamp",
|
|
||||||
},
|
|
||||||
DisableHTMLEscape: true,
|
|
||||||
})
|
|
||||||
},
|
|
||||||
Run: func(cmd *cobra.Command, args []string) {
|
|
||||||
debug.EnableDebugServer()
|
|
||||||
akURL := config.Get().AuthentikHost
|
|
||||||
if akURL == "" {
|
|
||||||
fmt.Println("env AUTHENTIK_HOST not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
akToken := config.Get().AuthentikToken
|
|
||||||
if akToken == "" {
|
|
||||||
fmt.Println("env AUTHENTIK_TOKEN not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
akURLActual, err := url.Parse(akURL)
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
fmt.Println(err)
|
|
||||||
fmt.Println(helpMessage)
|
fmt.Println(helpMessage)
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
ex := common.Init()
|
|
||||||
defer common.Defer()
|
|
||||||
go func() {
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
os.Exit(0)
|
|
||||||
}
|
|
||||||
}()
|
|
||||||
|
|
||||||
ac := ak.NewAPIController(*akURLActual, akToken)
|
|
||||||
if ac == nil {
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
defer ac.Shutdown()
|
|
||||||
|
|
||||||
ac.Server = ldap.NewServer(ac)
|
|
||||||
|
|
||||||
err = ac.Start()
|
|
||||||
if err != nil {
|
|
||||||
log.WithError(err).Panic("Failed to run server")
|
|
||||||
}
|
|
||||||
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
}
|
}
|
||||||
|
return err
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2,17 +2,13 @@ package main
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"fmt"
|
"fmt"
|
||||||
"net/url"
|
|
||||||
"os"
|
"os"
|
||||||
|
|
||||||
log "github.com/sirupsen/logrus"
|
|
||||||
"github.com/spf13/cobra"
|
"github.com/spf13/cobra"
|
||||||
|
|
||||||
"goauthentik.io/internal/common"
|
"goauthentik.io/internal/common"
|
||||||
"goauthentik.io/internal/config"
|
|
||||||
"goauthentik.io/internal/constants"
|
"goauthentik.io/internal/constants"
|
||||||
"goauthentik.io/internal/debug"
|
"goauthentik.io/internal/outpost/ak/entrypoint"
|
||||||
"goauthentik.io/internal/outpost/ak"
|
|
||||||
"goauthentik.io/internal/outpost/ak/healthcheck"
|
"goauthentik.io/internal/outpost/ak/healthcheck"
|
||||||
"goauthentik.io/internal/outpost/proxyv2"
|
"goauthentik.io/internal/outpost/proxyv2"
|
||||||
)
|
)
|
||||||
@ -28,65 +24,15 @@ Optionally, you can set these:
|
|||||||
- AUTHENTIK_HOST_BROWSER: URL to use in the browser, when it differs from AUTHENTIK_HOST`
|
- AUTHENTIK_HOST_BROWSER: URL to use in the browser, when it differs from AUTHENTIK_HOST`
|
||||||
|
|
||||||
var rootCmd = &cobra.Command{
|
var rootCmd = &cobra.Command{
|
||||||
Long: helpMessage,
|
Long: helpMessage,
|
||||||
Version: constants.FullVersion(),
|
Version: constants.FullVersion(),
|
||||||
PersistentPreRun: func(cmd *cobra.Command, args []string) {
|
PersistentPreRun: common.PreRun,
|
||||||
log.SetLevel(log.DebugLevel)
|
RunE: func(cmd *cobra.Command, args []string) error {
|
||||||
log.SetFormatter(&log.JSONFormatter{
|
err := entrypoint.OutpostMain("authentik.outpost.proxy", proxyv2.NewProxyServer)
|
||||||
FieldMap: log.FieldMap{
|
|
||||||
log.FieldKeyMsg: "event",
|
|
||||||
log.FieldKeyTime: "timestamp",
|
|
||||||
},
|
|
||||||
DisableHTMLEscape: true,
|
|
||||||
})
|
|
||||||
},
|
|
||||||
Run: func(cmd *cobra.Command, args []string) {
|
|
||||||
debug.EnableDebugServer()
|
|
||||||
akURL := config.Get().AuthentikHost
|
|
||||||
if akURL == "" {
|
|
||||||
fmt.Println("env AUTHENTIK_HOST not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
akToken := config.Get().AuthentikToken
|
|
||||||
if akToken == "" {
|
|
||||||
fmt.Println("env AUTHENTIK_TOKEN not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
akURLActual, err := url.Parse(akURL)
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
fmt.Println(err)
|
|
||||||
fmt.Println(helpMessage)
|
fmt.Println(helpMessage)
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
ex := common.Init()
|
|
||||||
defer common.Defer()
|
|
||||||
go func() {
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
os.Exit(0)
|
|
||||||
}
|
|
||||||
}()
|
|
||||||
|
|
||||||
ac := ak.NewAPIController(*akURLActual, akToken)
|
|
||||||
if ac == nil {
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
defer ac.Shutdown()
|
|
||||||
|
|
||||||
ac.Server = proxyv2.NewProxyServer(ac)
|
|
||||||
|
|
||||||
err = ac.Start()
|
|
||||||
if err != nil {
|
|
||||||
log.WithError(err).Panic("Failed to run server")
|
|
||||||
}
|
|
||||||
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
}
|
}
|
||||||
|
return err
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2,16 +2,13 @@ package main
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"fmt"
|
"fmt"
|
||||||
"net/url"
|
|
||||||
"os"
|
"os"
|
||||||
|
|
||||||
log "github.com/sirupsen/logrus"
|
|
||||||
"github.com/spf13/cobra"
|
"github.com/spf13/cobra"
|
||||||
|
|
||||||
"goauthentik.io/internal/common"
|
"goauthentik.io/internal/common"
|
||||||
"goauthentik.io/internal/constants"
|
"goauthentik.io/internal/constants"
|
||||||
"goauthentik.io/internal/debug"
|
"goauthentik.io/internal/outpost/ak/entrypoint"
|
||||||
"goauthentik.io/internal/outpost/ak"
|
|
||||||
"goauthentik.io/internal/outpost/ak/healthcheck"
|
"goauthentik.io/internal/outpost/ak/healthcheck"
|
||||||
"goauthentik.io/internal/outpost/rac"
|
"goauthentik.io/internal/outpost/rac"
|
||||||
)
|
)
|
||||||
@ -24,65 +21,15 @@ Required environment variables:
|
|||||||
- AUTHENTIK_INSECURE: Skip SSL Certificate verification`
|
- AUTHENTIK_INSECURE: Skip SSL Certificate verification`
|
||||||
|
|
||||||
var rootCmd = &cobra.Command{
|
var rootCmd = &cobra.Command{
|
||||||
Long: helpMessage,
|
Long: helpMessage,
|
||||||
Version: constants.FullVersion(),
|
Version: constants.FullVersion(),
|
||||||
PersistentPreRun: func(cmd *cobra.Command, args []string) {
|
PersistentPreRun: common.PreRun,
|
||||||
log.SetLevel(log.DebugLevel)
|
RunE: func(cmd *cobra.Command, args []string) error {
|
||||||
log.SetFormatter(&log.JSONFormatter{
|
err := entrypoint.OutpostMain("authentik.outpost.rac", rac.NewServer)
|
||||||
FieldMap: log.FieldMap{
|
|
||||||
log.FieldKeyMsg: "event",
|
|
||||||
log.FieldKeyTime: "timestamp",
|
|
||||||
},
|
|
||||||
DisableHTMLEscape: true,
|
|
||||||
})
|
|
||||||
},
|
|
||||||
Run: func(cmd *cobra.Command, args []string) {
|
|
||||||
debug.EnableDebugServer()
|
|
||||||
akURL, found := os.LookupEnv("AUTHENTIK_HOST")
|
|
||||||
if !found {
|
|
||||||
fmt.Println("env AUTHENTIK_HOST not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
akToken, found := os.LookupEnv("AUTHENTIK_TOKEN")
|
|
||||||
if !found {
|
|
||||||
fmt.Println("env AUTHENTIK_TOKEN not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
akURLActual, err := url.Parse(akURL)
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
fmt.Println(err)
|
|
||||||
fmt.Println(helpMessage)
|
fmt.Println(helpMessage)
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
ex := common.Init()
|
|
||||||
defer common.Defer()
|
|
||||||
go func() {
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
os.Exit(0)
|
|
||||||
}
|
|
||||||
}()
|
|
||||||
|
|
||||||
ac := ak.NewAPIController(*akURLActual, akToken)
|
|
||||||
if ac == nil {
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
defer ac.Shutdown()
|
|
||||||
|
|
||||||
ac.Server = rac.NewServer(ac)
|
|
||||||
|
|
||||||
err = ac.Start()
|
|
||||||
if err != nil {
|
|
||||||
log.WithError(err).Panic("Failed to run server")
|
|
||||||
}
|
|
||||||
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
}
|
}
|
||||||
|
return err
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2,16 +2,13 @@ package main
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"fmt"
|
"fmt"
|
||||||
"net/url"
|
|
||||||
"os"
|
"os"
|
||||||
|
|
||||||
log "github.com/sirupsen/logrus"
|
|
||||||
"github.com/spf13/cobra"
|
"github.com/spf13/cobra"
|
||||||
|
|
||||||
"goauthentik.io/internal/common"
|
"goauthentik.io/internal/common"
|
||||||
"goauthentik.io/internal/constants"
|
"goauthentik.io/internal/constants"
|
||||||
"goauthentik.io/internal/debug"
|
"goauthentik.io/internal/outpost/ak/entrypoint"
|
||||||
"goauthentik.io/internal/outpost/ak"
|
|
||||||
"goauthentik.io/internal/outpost/ak/healthcheck"
|
"goauthentik.io/internal/outpost/ak/healthcheck"
|
||||||
"goauthentik.io/internal/outpost/radius"
|
"goauthentik.io/internal/outpost/radius"
|
||||||
)
|
)
|
||||||
@ -24,65 +21,15 @@ Required environment variables:
|
|||||||
- AUTHENTIK_INSECURE: Skip SSL Certificate verification`
|
- AUTHENTIK_INSECURE: Skip SSL Certificate verification`
|
||||||
|
|
||||||
var rootCmd = &cobra.Command{
|
var rootCmd = &cobra.Command{
|
||||||
Long: helpMessage,
|
Long: helpMessage,
|
||||||
Version: constants.FullVersion(),
|
Version: constants.FullVersion(),
|
||||||
PersistentPreRun: func(cmd *cobra.Command, args []string) {
|
PersistentPreRun: common.PreRun,
|
||||||
log.SetLevel(log.DebugLevel)
|
RunE: func(cmd *cobra.Command, args []string) error {
|
||||||
log.SetFormatter(&log.JSONFormatter{
|
err := entrypoint.OutpostMain("authentik.outpost.radius", radius.NewServer)
|
||||||
FieldMap: log.FieldMap{
|
|
||||||
log.FieldKeyMsg: "event",
|
|
||||||
log.FieldKeyTime: "timestamp",
|
|
||||||
},
|
|
||||||
DisableHTMLEscape: true,
|
|
||||||
})
|
|
||||||
},
|
|
||||||
Run: func(cmd *cobra.Command, args []string) {
|
|
||||||
debug.EnableDebugServer()
|
|
||||||
akURL, found := os.LookupEnv("AUTHENTIK_HOST")
|
|
||||||
if !found {
|
|
||||||
fmt.Println("env AUTHENTIK_HOST not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
akToken, found := os.LookupEnv("AUTHENTIK_TOKEN")
|
|
||||||
if !found {
|
|
||||||
fmt.Println("env AUTHENTIK_TOKEN not set!")
|
|
||||||
fmt.Println(helpMessage)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
akURLActual, err := url.Parse(akURL)
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
fmt.Println(err)
|
|
||||||
fmt.Println(helpMessage)
|
fmt.Println(helpMessage)
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
ex := common.Init()
|
|
||||||
defer common.Defer()
|
|
||||||
go func() {
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
os.Exit(0)
|
|
||||||
}
|
|
||||||
}()
|
|
||||||
|
|
||||||
ac := ak.NewAPIController(*akURLActual, akToken)
|
|
||||||
if ac == nil {
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
defer ac.Shutdown()
|
|
||||||
|
|
||||||
ac.Server = radius.NewServer(ac)
|
|
||||||
|
|
||||||
err = ac.Start()
|
|
||||||
if err != nil {
|
|
||||||
log.WithError(err).Panic("Failed to run server")
|
|
||||||
}
|
|
||||||
|
|
||||||
for {
|
|
||||||
<-ex
|
|
||||||
}
|
}
|
||||||
|
return err
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Some files were not shown because too many files have changed in this diff Show More
Reference in New Issue
Block a user