Skip to content

providers/oauth2: cleanup tokens when user is deactivated#12859

Merged
rissson merged 4 commits intomainfrom
fix-12858
Feb 12, 2025
Merged

providers/oauth2: cleanup tokens when user is deactivated#12859
rissson merged 4 commits intomainfrom
fix-12858

Conversation

@rissson
Copy link
Member

@rissson rissson commented Jan 28, 2025

Details

Closes #12858


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 website)

Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
@rissson rissson self-assigned this Jan 28, 2025
@rissson rissson requested a review from a team as a code owner January 28, 2025 12:26
@netlify
Copy link

netlify bot commented Jan 28, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit 5339525
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/67acb4c2c981c8000899c008

@netlify
Copy link

netlify bot commented Jan 28, 2025

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit 5339525
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/67acb4c21bfa60000825ae0a
😎 Deploy Preview https://deploy-preview-12859--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 site configuration.

@codecov
Copy link

codecov bot commented Jan 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.76%. Comparing base (141ced8) to head (5339525).
Report is 69 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12859      +/-   ##
==========================================
+ Coverage   92.69%   92.76%   +0.06%     
==========================================
  Files         769      785      +16     
  Lines       38926    39592     +666     
==========================================
+ Hits        36083    36728     +645     
- Misses       2843     2864      +21     
Flag Coverage Δ
e2e 48.56% <100.00%> (+0.08%) ⬆️
integration 24.54% <66.66%> (-0.06%) ⬇️
unit 90.43% <100.00%> (+0.03%) ⬆️

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 28, 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-5339525cd0cfff4652ecb7b0ec9c92f8199b4b9a
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-5339525cd0cfff4652ecb7b0ec9c92f8199b4b9a

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

rissson and others added 3 commits January 28, 2025 17:07
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@rissson rissson enabled auto-merge (squash) February 12, 2025 14:57
@rissson rissson merged commit ac4d6ae into main Feb 12, 2025
70 checks passed
@rissson rissson deleted the fix-12858 branch February 12, 2025 15:13
kensternberg-authentik added a commit that referenced this pull request Feb 20, 2025
* main: (111 commits)
  root: correctly use correct schema for install_id (#13018)
  website: bump docusaurus-plugin-openapi-docs from 4.3.3 to 4.3.4 in /website (#13011)
  web: bump API Client version (#13017)
  core: bump aws-cdk-lib from 2.178.1 to 2.178.2 (#13013)
  core: bump oss/go/microsoft/golang from 1.23-fips-bookworm to 1.24-fips-bookworm (#13012)
  website: bump docusaurus-theme-openapi-docs from 4.3.3 to 4.3.4 in /website (#13010)
  lifecycle/aws: bump aws-cdk from 2.178.1 to 2.178.2 in /lifecycle/aws (#13009)
  core: bump github.com/sethvargo/go-envconfig from 1.1.0 to 1.1.1 (#13008)
  web/admin: fix source selection for identification stage (#13007)
  core: bump sentry-sdk from 2.20.0 to 2.21.0 (#13014)
  website/integrations: Open WebUI (#12939)
  root: use correct default schema for install_id (#13006)
  website/docs: fix a minor typo (#13004)
  enterprise/providers/ssf: fixes v2 (#13003)
  root: make default postgres schema configurable (#12949)
  providers/oauth2: cleanup tokens when user is deactivated (#12859)
  website/docs: fix Nginx redirection example (#12920)
  core: bump twilio from 9.4.4 to 9.4.5 (#12993)
  core: bump coverage from 7.6.11 to 7.6.12 (#12994)
  core: bump cryptography from 44.0.0 to 44.0.1 (#12992)
  ...
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.

user refresh tokens still active after user deactivation.

2 participants