Skip to content

admin/files: add check for /media existence#18636

Merged
rissson merged 6 commits intomainfrom
files-add-check
Dec 10, 2025
Merged

admin/files: add check for /media existence#18636
rissson merged 6 commits intomainfrom
files-add-check

Conversation

@rissson
Copy link
Member

@rissson rissson commented Dec 5, 2025

Details

REPLACE ME


Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make docs)

Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson self-assigned this Dec 5, 2025
@rissson rissson requested a review from a team as a code owner December 5, 2025 17:49
@netlify
Copy link

netlify bot commented Dec 5, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 70de65f
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/6939779f0304e20008ac40f9

@rissson rissson added this to the Release 2025.12 milestone Dec 5, 2025
@rissson rissson moved this from Todo to Needs review in authentik Core Dec 5, 2025
@netlify
Copy link

netlify bot commented Dec 5, 2025

Deploy Preview for authentik-integrations canceled.

Name Link
🔨 Latest commit 70de65f
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/6939779f9cef5c00089aa0ad

@netlify
Copy link

netlify bot commented Dec 5, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit 70de65f
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/6939779f8397ab0008c1f71f

@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.22%. Comparing base (01ed831) to head (70de65f).
⚠️ Report is 25 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #18636      +/-   ##
==========================================
- Coverage   93.24%   93.22%   -0.03%     
==========================================
  Files         941      944       +3     
  Lines       51561    51672     +111     
==========================================
+ Hits        48078    48169      +91     
- Misses       3483     3503      +20     
Flag Coverage Δ
e2e 44.75% <ø> (-0.04%) ⬇️
integration 23.29% <ø> (+0.01%) ⬆️
unit 91.45% <ø> (-0.02%) ⬇️
unit-migrate 91.50% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-70de65f4b38901d941924f1480e0c74a3164e986
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-70de65f4b38901d941924f1480e0c74a3164e986

Afterwards, run the upgrade commands from the latest release notes.

Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson requested a review from dominic-r December 9, 2025 16:58
Copy link
Member

@dominic-r dominic-r left a comment

Choose a reason for hiding this comment

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

mabye linking release notes to be precise, but LGTM

Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson enabled auto-merge (squash) December 10, 2025 13:37
@rissson rissson merged commit 54be518 into main Dec 10, 2025
136 of 138 checks passed
@rissson rissson deleted the files-add-check branch December 10, 2025 14:56
@github-project-automation github-project-automation bot moved this from Needs review to Done in authentik Core Dec 10, 2025
rissson added a commit that referenced this pull request Dec 15, 2025
rissson added a commit that referenced this pull request Dec 15, 2025
…ick #18829 to version-2025.12) (#18838)

Co-authored-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
kensternberg-authentik added a commit that referenced this pull request Dec 18, 2025
* main: (54 commits)
  website/docs: 2025.10.3 release notes (#18868)
  website/docs: Add docs for passkey autofill (WebauthN Conditional UI) (#18805)
  website/docs: adjust RBAC-related details in 2025.12 release notes (#18863)
  outposts: fix permission errors for related certificates (#18861)
  web/admin/rbac: misc object permission fixes (#18859)
  core: bump library/golang from `5d35fb8` to `8e8f9c8` (#18855)
  rbac: alter migrated direct permission roles (#18860)
  core: add skip s3_test_server_available to TestResolveFileUrlS3Backend (#18858)
  ci: replace codecov test-results action (#18862)
  core: bump goauthentik/fips-debian from `c10cd2c` to `2f19fc1` (#18856)
  admin/files: fix get_objects_for_user queryset argument in FileUsedByView (#18845)
  core: skip s3 tests if endpoint isn't available (#18841)
  crypto: Store details parsed from includeDetails in database instead (#18013)
  website/docs: add jellyseer integration doc (#18812)
  admin/files: revert add check for /media existence (#18636) (#18829)
  core: bump goauthentik.io/api/v3 from 3.2025120.26 to 3.2026020.1 (#18815)
  packages/django-dramatiq-postgres: broker: close django connections on consumer close (#18833)
  core: remove superuser check from `Token` list (#18684)
  website/docs: add icon info to style guide (#18832)
  core: list applications fix (#18798)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants