fix(windows): licentie-pagina, bestandsassociaties en handtekening-upload (#1600) #1621
No reviewers
Labels
No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
LibreKAT/Ocideck!1621
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/windows-installer-restpunten-1600"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Samenvatting
Drie punten uit #1600 die de Windows-installatie-ervaring en de release-veiligheid verbeteren:
1. Licentie-pagina toont platte tekst, geen ruwe Markdown
Inno Setup's
LicenseFiletoont de inhoud letterlijk — de HTML-commentaar,#/##/**/>markers en---regels uitLICENSE.mdstonden op het eerste scherm dat elke Windows-gebruiker ziet.tool/generate_license_txt.dartstripst Markdown naar platte tekst.packaging/windows/LICENSE.txtis de gegenereerde uitvoer (gecommit, want de installer bouwt offline)..isswijst naarLICENSE.txti.p.v.LICENSE.md.LICENSE.md-edit zonder re-run faalt de poort.2. Bestandsassociaties gebruiken een placeholder, niet
C:\Program FilesDe
.regzei "per gebruiker, geen beheerdersrechten nodig" maar hardcodeerdeC:\Program Files\OciDeck— een pad dat admin-rechten vereist en de belofte tegenspreekt. Nu staat<JOUW-OCIDECK-MAP>er: een obvious placeholder, niet een echt-uitziend default-pad.3. Release-script uploadt de handtekening veilig (upload-then-replace)
De vorige volgorde (DELETE dan POST) liet bij een gefaalde POST de release zonder handtekening achter; een stil gefaalde DELETE (
|| true) leidde tot een 409 op de POST. Nu: POST onder.new→ DELETE oude → PATCH hernoem. Faalt de POST, dan staat de oude nog op de release.Testplan
test/windows_packaging_test.dart— 3 nieuwe tests (licentie .txt bestaat, is vers, bevat geen Markdown) + 1 nieuwe test (.reg placeholder)test/release_auto_version_test.dart— test ververst voor upload-then-replace volgordemake check-staticgroenmake check-secrets— geen leaks (gitleaks + trufflehog)make sast— 0 findings (semgrep)origin/main(5 commits)Generated with Devin