Skip to content

packages/ak-common/tracing: make console subscriber optional#24219

Open
rissson wants to merge 1 commit into
mainfrom
console-subscriber-optional
Open

packages/ak-common/tracing: make console subscriber optional#24219
rissson wants to merge 1 commit into
mainfrom
console-subscriber-optional

Conversation

@rissson

@rissson rissson commented Jul 20, 2026

Copy link
Copy Markdown
Member

Details

What does this PR change?

Make listening on debug_tokio optional.

Why is this change needed?

Currently we always try to listen there and fail when the port is already occupied, for example if another authentik process is running. So running both the server and proxy outpost (for example) in dev would crash the second process that starts.

How was this tested?

Linked issues


Checklist

  • The project has been linted, built, and tested (make all)
  • The documentation has been updated and formatted (make docs)

Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson self-assigned this Jul 20, 2026
@rissson
rissson requested a review from a team as a code owner July 20, 2026 12:42
@netlify

netlify Bot commented Jul 20, 2026

Copy link
Copy Markdown

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 98d6c5f
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/6a5e17ab78a244000829c2e3
😎 Deploy Preview https://deploy-preview-24219--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.47%. Comparing base (f0d61a4) to head (98d6c5f).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #24219      +/-   ##
==========================================
- Coverage   93.50%   93.47%   -0.04%     
==========================================
  Files        1060     1060              
  Lines       63630    63630              
  Branches      424      424              
==========================================
- Hits        59498    59478      -20     
- Misses       4132     4152      +20     
Flag Coverage Δ
conformance 35.51% <ø> (+<0.01%) ⬆️
e2e 40.48% <ø> (+<0.01%) ⬆️
integration 31.97% <ø> (-0.05%) ⬇️
rust 0.00% <ø> (ø)
unit 92.68% <ø> (+<0.01%) ⬆️
unit-migrate 92.71% <ø> (+<0.01%) ⬆️

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

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

@rissson
rissson enabled auto-merge (squash) July 20, 2026 13:09
@github-actions

Copy link
Copy Markdown
Contributor

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-98d6c5f9c8a614f7b09d93b1f436896c500cca82
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-98d6c5f9c8a614f7b09d93b1f436896c500cca82

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

@rissson rissson moved this from Todo to Needs review in authentik Core Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Needs review

Development

Successfully merging this pull request may close these issues.

1 participant