MSIX storage validation leaves the sideload certificate in TrustedPeople

Abierto Apto para principiantes
  • ¿Algo no está bien?
#2,009 2 comentarios 1 reacción 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
77/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
powershell
Área
devops

Línea de trabajo

Lee scripts/validate-msix-storage-paths.ps1 en torno a la importación del certificado en la línea 591 y Invoke-Teardown en torno a la línea 1041. Registra si esta ejecución añadió el certificado y, durante la limpieza, elimina esa huella digital solo si se importó por primera vez. Añade una regresión o una aserción en el script que muestre que un certificado que ya estaba en TrustedPeople permanece allí.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

clawsweeper:bulk-filed clawsweeper:needs-security-review clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:security issue-rating: 🦞 diamond lobster P2

What happens

scripts/validate-msix-storage-paths.ps1 -CertPath imports that certificate into Cert:\LocalMachine\TrustedPeople so the sideload package can install. Teardown removes the AppX package with Remove-AppxPackage and does not remove the certificate. The publisher stays trusted on the machine after the validation run exits.

Import requires an elevated store write. The leftover is the trust entry the script added and then did not undo.

Where

Checked on 7967d509 by reading the script. The script was not run.

Suggested fix

Record the imported thumbprint and remove that certificate from TrustedPeople during teardown. Do not remove a different certificate that was already in the store.

Environment

Windows 11. OpenClaw Companion source on main at commit 7967d509886fc9efecbb19c8236350a82867f69c (2026-11-04). No packaged installer build was used.

Steps

  1. On Windows, as an administrator, run scripts/validate-msix-storage-paths.ps1 -CertPath with a test .cer.
  2. Let the script import the certificate and finish teardown.
  3. Open Cert:\LocalMachine\TrustedPeople and find that thumbprint.

Line 591 calls Import-Certificate into Cert:\LocalMachine\TrustedPeople. Invoke-Teardown removes the AppX package and does not remove the certificate.

Repair

Remember the thumbprint this run imported and remove only that certificate during teardown. Add a regression, or a script assertion, that a thumbprint which was already in the store is left in place. Stop there.

Lenguaje dominante
C#
Estrellas
2.2k
Forks
340
Merge medio
3 d 11 h
PR fusionados (30 d)
173

Preparar el entorno

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de openclaw/openclaw-windows-node

Todos los issues de openclaw/openclaw-windows-node

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.