Skip to content

fix: windows server 2019 activation failing because of incomplete entry in KmsDataBase.xml#143

Merged
simonmicro merged 2 commits intoPy-KMS-Organization:nextfrom
Rileran:fix/windows-server-2019
Apr 17, 2026
Merged

fix: windows server 2019 activation failing because of incomplete entry in KmsDataBase.xml#143
simonmicro merged 2 commits intoPy-KMS-Organization:nextfrom
Rileran:fix/windows-server-2019

Conversation

@Rileran
Copy link
Copy Markdown

@Rileran Rileran commented Jan 22, 2026

This fixes issue #138, where Windows Server 2019 would not work because of an entry in KmsDataBase.xml.

The entry for Windows Server 2019 (Azure Only) is missing MinKeyId and MaxKeyId which are required for pykms to compute the pid.

		<CsvlkItem DisplayName="Windows Server 2019 (Azure Only)" ReleaseDate="2018-10-02T00:00:00Z" GroupId="206" IniFileName="Windows" Id="3c006fa7-3b03-45a4-93da-63ddc1bdce11">
			<Activate KmsItem="8449b1fb-f0ea-497a-99ab-66ca96e9a0f5" />
			<Activate KmsItem="cbdcb1ba-e093-4c81-a463-10775291fdf9" />
		</CsvlkItem>

What this PR does is ignoring any invalid entries in the XML file, preventing the pid_generator from not returning when other valid entries exists.

@Rileran Rileran force-pushed the fix/windows-server-2019 branch from 92f904b to ecf50e0 Compare January 22, 2026 14:54
MrRubberDucky added a commit to Rubberverse/qor-kms that referenced this pull request Feb 26, 2026
this totally will go well and won't break everything.

Signed-off-by: Mr. Rubber Ducky <79613254+MrRubberDucky@users.noreply.github.com>
@simonmicro simonmicro self-assigned this Apr 11, 2026
@simonmicro
Copy link
Copy Markdown
Member

simonmicro commented Apr 11, 2026

Could you please target the next branch? I'll then happily include it with the next release

Ah, I found the base-branch button again after the UI change by Github. Lol

@simonmicro simonmicro changed the base branch from master to next April 17, 2026 08:26
@simonmicro simonmicro merged commit 8dea676 into Py-KMS-Organization:next Apr 17, 2026
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.

2 participants