Skip to content

web: Enforce challenge nullish types.#19768

Merged
GirlBossRush merged 2 commits intomainfrom
enforce-challange-property-types
Jan 27, 2026
Merged

web: Enforce challenge nullish types.#19768
GirlBossRush merged 2 commits intomainfrom
enforce-challange-property-types

Conversation

@GirlBossRush
Copy link
Contributor

@GirlBossRush GirlBossRush commented Jan 26, 2026

Details

This PR enforces the actual type of the challenge property in flow stages, i.e. consistently checking for nullish values.

Closes: #19774

@GirlBossRush GirlBossRush added this to the Release 2025.12.2 milestone Jan 26, 2026
@GirlBossRush GirlBossRush self-assigned this Jan 26, 2026
@GirlBossRush GirlBossRush requested a review from a team as a code owner January 26, 2026 18:08
@GirlBossRush GirlBossRush added area:frontend Features or issues related to the browser, TypeScript, Node.js, etc backport/version-2025.12 Add this label to PRs to backport changes to version-2025.12 labels Jan 26, 2026
@netlify
Copy link

netlify bot commented Jan 26, 2026

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit cc49cce
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/697804ff6edc2b0008e0462a
😎 Deploy Preview https://deploy-preview-19768--authentik-storybook.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
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #19768      +/-   ##
==========================================
+ Coverage   92.99%   93.22%   +0.23%     
==========================================
  Files         954      954              
  Lines       52443    52443              
==========================================
+ Hits        48768    48891     +123     
+ Misses       3675     3552     -123     
Flag Coverage Δ
conformance 38.23% <ø> (+<0.01%) ⬆️
e2e 44.08% <ø> (+0.61%) ⬆️
integration 23.13% <ø> (+<0.01%) ⬆️
unit 91.48% <ø> (+<0.01%) ⬆️
unit-migrate 91.49% <ø> (-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 Jan 26, 2026

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-cc49cceccbc8f91383cbe7fecb68afbcda74773f
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-cc49cceccbc8f91383cbe7fecb68afbcda74773f

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

Copy link
Contributor

@kensternberg-authentik kensternberg-authentik left a comment

Choose a reason for hiding this comment

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

Routine. I've seen a lot of shotgun surgery already this week, and it's already Monday. What can we do to bring these patterns into code?

@GirlBossRush GirlBossRush force-pushed the enforce-challange-property-types branch from c847a88 to 276e8d0 Compare January 27, 2026 00:10
@netlify
Copy link

netlify bot commented Jan 27, 2026

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 276e8d0
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/697802813096a90008f296cb
😎 Deploy Preview https://deploy-preview-19768--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.

@GirlBossRush GirlBossRush merged commit f080a82 into main Jan 27, 2026
101 checks passed
@GirlBossRush GirlBossRush deleted the enforce-challange-property-types branch January 27, 2026 01:08
authentik-automation bot pushed a commit that referenced this pull request Jan 27, 2026
This cherry-pick has conflicts that need manual resolution.

Original PR: #19768
Original commit: f080a82
@authentik-automation
Copy link
Contributor

⚠️ Cherry-pick to version-2025.12 has conflicts: #19777

GirlBossRush added a commit that referenced this pull request Jan 27, 2026
This cherry-pick has conflicts that need manual resolution.

Original PR: #19768
Original commit: f080a82
GirlBossRush added a commit that referenced this pull request Jan 27, 2026
This cherry-pick has conflicts that need manual resolution.

Original PR: #19768
Original commit: f080a82
GirlBossRush added a commit that referenced this pull request Jan 27, 2026
This cherry-pick has conflicts that need manual resolution.

Original PR: #19768
Original commit: f080a82
GirlBossRush added a commit that referenced this pull request Jan 30, 2026
This cherry-pick has conflicts that need manual resolution.

Original PR: #19768
Original commit: f080a82
melizeche added a commit that referenced this pull request Jan 30, 2026
…n-2025.12) (#19777)

* Cherry-pick #19768 to version-2025.12 (with conflicts)

This cherry-pick has conflicts that need manual resolution.

Original PR: #19768
Original commit: f080a82

* Fix type.

---------

Co-authored-by: Teffen Ellis <592134+GirlBossRush@users.noreply.github.com>
Co-authored-by: Marcelo Elizeche Landó <marcelo@goauthentik.io>
kensternberg-authentik added a commit that referenced this pull request Jan 31, 2026
* main: (45 commits)
  sources/saml: Add testcases for PR #19593 (#19647)
  revert: website/integrations: wazuh: Change exchange key generation to 64 bytes (#19759)
  web: bump API Client version (#19760)
  core: bump djangoql from 0.18.2 to 0.19.1 (#19780)
  web: Vendor SFE Bootstrap (#19766)
  core, web: update translations (#19717)
  web: bump the eslint group across 1 directory with 3 updates (#19782)
  web: bump the react group across 1 directory with 2 updates (#19784)
  web: bump country-flag-icons from 1.6.8 to 1.6.9 in /web (#19785)
  providers/oauth2: Support login_hint (#19498)
  admin/files: add centralized theme variable support for file URLs (#19657)
  core: bump github.com/pires/go-proxyproto from 0.9.1 to 0.9.2 (#19778)
  core: bump openapitools/openapi-diff from 2.1.6 to 2.1.7 in /scripts/api (#19779)
  core: bump gssapi from 1.10.1 to 1.11.1 (#19781)
  ci: bump actions/attest-build-provenance from 3.1.0 to 3.2.0 (#19783)
  website/docs: endpoint devices: fix local device login (#19698)
  web: Enforce `challenge` nullish types. (#19768)
  web/elements: stabilize dual-select status height (#19734)
  web/a11y: CAPTCHA Stage Form (#19670)
  web/table: align row action icons and tooltip color (#19736)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Features or issues related to the browser, TypeScript, Node.js, etc backport/version-2025.12 Add this label to PRs to backport changes to version-2025.12

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding second Email-MFA adds strange "}" on bottom of the form

3 participants