Skip to content

fix(wireguard-gui): polkit regex to match wayland-ghaf display name#1860

Merged
brianmcgillion merged 1 commit intotiiuae:mainfrom
enesoztrk:fix/wireguard-gui-polkit
Mar 27, 2026
Merged

fix(wireguard-gui): polkit regex to match wayland-ghaf display name#1860
brianmcgillion merged 1 commit intotiiuae:mainfrom
enesoztrk:fix/wireguard-gui-polkit

Conversation

@enesoztrk
Copy link
Copy Markdown
Contributor

@enesoztrk enesoztrk commented Mar 27, 2026

Description of Changes

The previous regex expected exactly 10 alphanumeric characters after "wayland-" and used a fixed-length slice, which did not match the "wayland-ghaf" new display name used in Ghaf

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

SSRCSP-8185

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other:

Test Steps To Verify:

  1. Try to launch WireGuard GUI apps in enabled VMs.

Signed-off-by: Enes Öztürk <enes.ozturk@unikie.com>
@enesoztrk enesoztrk added the Needs Testing CI Team to pre-verify label Mar 27, 2026
@enesoztrk enesoztrk requested review from mbssrc and vunnyso March 27, 2026 11:11
@milva-unikie
Copy link
Copy Markdown

Tested on Darter Pro

  • Wireguard GUI opens again from the Ghaf Control Panel

@milva-unikie milva-unikie added Tested on System76 and removed Needs Testing CI Team to pre-verify labels Mar 27, 2026
@brianmcgillion brianmcgillion merged commit a48f6be into tiiuae:main Mar 27, 2026
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants