Skip to content

tasks: set uid early#17356

Merged
rissson merged 3 commits intomainfrom
tasks-eagerly-save-messages
Oct 9, 2025
Merged

tasks: set uid early#17356
rissson merged 3 commits intomainfrom
tasks-eagerly-save-messages

Conversation

@rissson
Copy link
Member

@rissson rissson commented Oct 9, 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)

@rissson rissson self-assigned this Oct 9, 2025
@rissson rissson requested a review from a team as a code owner October 9, 2025 11:55
@netlify
Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit f98da92
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/68e7b54b6157490008946368

@netlify
Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit f98da92
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/68e7b54b06d38a0008ba94da

@netlify
Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for authentik-integrations canceled.

Name Link
🔨 Latest commit f98da92
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/68e7b54b5ece310008a5528f

@codecov
Copy link

codecov bot commented Oct 9, 2025

Codecov Report

❌ Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.92%. Comparing base (4777bbd) to head (f98da92).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
authentik/lib/sync/outgoing/api.py 40.00% 3 Missing ⚠️
authentik/outposts/signals.py 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #17356      +/-   ##
==========================================
- Coverage   92.97%   92.92%   -0.06%     
==========================================
  Files         857      857              
  Lines       46563    46597      +34     
==========================================
+ Hits        43290    43298       +8     
- Misses       3273     3299      +26     
Flag Coverage Δ
e2e 45.91% <41.66%> (-0.13%) ⬇️
integration 23.11% <33.33%> (-0.01%) ⬇️
unit 91.04% <79.16%> (+<0.01%) ⬆️
unit-migrate 91.10% <79.16%> (+<0.01%) ⬆️

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.

@rissson
Copy link
Member Author

rissson commented Oct 9, 2025

yeah so this is not working

@rissson rissson marked this pull request as draft October 9, 2025 12:19
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson force-pushed the tasks-eagerly-save-messages branch from c591c02 to c2632b5 Compare October 9, 2025 12:58
@rissson rissson changed the title tasks: eagerly save uid and messages tasks: set uid early Oct 9, 2025
@rissson rissson marked this pull request as ready for review October 9, 2025 12:59
@rissson rissson requested a review from BeryJu October 9, 2025 12:59
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson enabled auto-merge (squash) October 9, 2025 13:00
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson merged commit 6341252 into main Oct 9, 2025
94 of 97 checks passed
@rissson rissson deleted the tasks-eagerly-save-messages branch October 9, 2025 13:41
kensternberg-authentik added a commit that referenced this pull request Nov 10, 2025
* main: (56 commits)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#17361)
  website/docs: add entra id scim source (#17357)
  outpost: proxyv2: Use Postgres for the Embedded Outpost (#16628)
  tasks: set uid early (#17356)
  tasks: only set tenant on task creation (#17358)
  core: bump golang.org/x/oauth2 from 0.31.0 to 0.32.0 (#17346)
  web: bump eslint-plugin-react-hooks from 6.1.1 to 7.0.0 in /packages/eslint-config in the eslint group across 1 directory (#17347)
  web: bump chromedriver from 141.0.0 to 141.0.1 in /web (#17348)
  web: bump knip from 5.64.1 to 5.64.2 in /web (#17349)
  web: bump @formatjs/intl-listformat from 7.7.11 to 7.7.12 in /web (#17350)
  web: bump pino from 9.13.0 to 10.0.0 in /web (#17351)
  core: bump axllent/mailpit from v1.27.9 to v1.27.10 in /tests/e2e (#17352)
  enterprise/providers/gws+entra: fix group integrity error during discovery (#17355)
  core, web: update translations (#17342)
  ci: bump snok/container-retention-policy from 2.2.1 to 3.0.1 (#17344)
  core: bump goauthentik.io/api/v3 from 3.2025100.18 to 3.2025100.20 (#17345)
  packages/django-dramatiq-postgres: broker: task retrieval fixes and improvements (#17335)
  enterprise/providers/gws+entra: fix integrity error during discovery (#17341)
  web: bump API Client version (#17340)
  api: Clean schema up more (#17055)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants