Skip to content

website/integrations: wazuh: Change exchange key generation to 64 bytes#18769

Merged
dominic-r merged 1 commit intogoauthentik:mainfrom
sirofwalls:patch-3
Dec 12, 2025
Merged

website/integrations: wazuh: Change exchange key generation to 64 bytes#18769
dominic-r merged 1 commit intogoauthentik:mainfrom
sirofwalls:patch-3

Conversation

@sirofwalls
Copy link
Contributor

@sirofwalls sirofwalls commented Dec 12, 2025

Updated the command to generate an exchange key from 32 to 64 bytes. as of wazuh 4.9 the exchange key needs to be 64 characters long

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)

@sirofwalls
Change exchange key generation to 64 bytes
c606eee 
Updated the command to generate an exchange key from 32 to 64 bytes. as of wazuh 4.9 the exchange key needs to be 64 characters long 

Signed-off-by: Sirofwalls <58705178+sirofwalls@users.noreply.github.com>
@sirofwalls sirofwalls requested a review from a team as a code owner December 12, 2025 03:45
@netlify
Copy link

netlify bot commented Dec 12, 2025 •
edited
Loading

✅ Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit c606eee
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/693b8fc7b0e8340008b6ac1a
😎 Deploy Preview https://deploy-preview-18769--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.

@netlify
Copy link

netlify bot commented Dec 12, 2025 •
edited
Loading

✅ Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit c606eee
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/693b8fc74158da0008dfa6bd
😎 Deploy Preview https://deploy-preview-18769--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.

@netlify
Copy link

netlify bot commented Dec 12, 2025 •
edited
Loading

✅ Deploy Preview for authentik-integrations ready!

Name Link
🔨 Latest commit c606eee
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/693b8fc792e35300083bf2ae
😎 Deploy Preview https://deploy-preview-18769--authentik-integrations.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.

@dominic-r dominic-r changed the title Change exchange key generation to 64 bytes website/integrations: wazuh: Change exchange key generation to 64 bytes Dec 12, 2025
dominic-r
dominic-r approved these changes Dec 12, 2025
View reviewed changes
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.

Thanks!

@dominic-r dominic-r self-assigned this Dec 12, 2025
@dominic-r dominic-r added the area:docs Features or issues related to Docusaurus label Dec 12, 2025
@dominic-r dominic-r enabled auto-merge (squash) December 12, 2025 03:47
@github-project-automation github-project-automation bot moved this from Todo to In Progress in authentik Core Dec 12, 2025
@codecov
Copy link

codecov bot commented Dec 12, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.18%. Comparing base (3d9632c) to head (c606eee).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #18769      +/-   ##
==========================================
+ Coverage   92.46%   93.18%   +0.72%     
==========================================
  Files         946      946              
  Lines       51862    51862              
==========================================
+ Hits        47954    48330     +376     
+ Misses       3908     3532     -376
Flag Coverage Δ
e2e 44.79% <ø> (+0.01%) ⬆️
integration 23.23% <ø> (?)
unit 91.47% <ø> (+<0.01%) ⬆️
unit-migrate 91.51% <ø> (+10.58%) ⬆️

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.

@dominic-r dominic-r merged commit cafe9e3 into goauthentik:main Dec 12, 2025
135 of 137 checks passed
kensternberg-authentik added a commit that referenced this pull request Dec 12, 2025
@kensternberg-authentik
Merge branch 'main' into admin-configurable-empty-table-message
aa41e5e 
* main:
  web/admin: fix typo in PolicyAccessView (#18789)
  translate: Updates for project authentik and language fr_FR (#18788)
  core: bump goauthentik/fips-debian from `dea09c4` to `07f41ce` (#18778)
  admin/files: cache expensive generated URLs (#18784)
  ci: bump actions/cache from 4.3.0 to 5.0.0 (#18779)
  ci: bump tj-actions/changed-files from 47.0.0 to 47.0.1 (#18780)
  core: bump goauthentik.io/api/v3 from 3.2025120.25 to 3.2025120.26 (#18770)
  core: bump selenium/standalone-chromium from 142.0 to 143.0 in /tests/e2e (#18772)
  web: bump types (merge branch) (#18735)
  ci: generate typescript api client for release too (#18761)
  website/integrations: wazuh: Change exchange key generation to 64 bytes (#18769)
@shellixa
Copy link

shellixa commented Jan 26, 2026

This pull request needs to be rolled back! The previous command with openssl rand -hex 32 was the correct one!
openssl rand -hex 64 will generate a string that is 64 bytes long but actually 128 characters! Because in hexadecimal format, 1 byte = 2 characters !
If you run:
openssl rand -hex 64 | wc -c the character count is 129 (128 characters + string terminator).

What you actually want is:
openssl rand -hex 32 | wc -c --> 65 (64 characters + string terminator)

@dewi-tik dewi-tik mentioned this pull request Jan 26, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dominic-r dominic-r dominic-r approved these changes

Assignees

@dominic-r dominic-r

Labels

area:docs Features or issues related to Docusaurus

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sirofwalls @shellixa @dominic-r @dewi-tik