Skip to content

Commit 7b6ee7f

Browse files
Merge b9ef7b6 into f6c8c9d
2 parents f6c8c9d + b9ef7b6 commit 7b6ee7f

2 files changed

Lines changed: 10 additions & 6 deletions

File tree

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
- 1.2.3
2+
- Fixed Sans Being passed through Extensions Data, Google does not like this.
13
- 1.2.2
24
- Fixed Sync Issues at CA Level, was ignoring and always syncing at pool level
35
- 1.2.1

GCPCAS/Client/CreateCertificateRequestBuilder.cs

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
Copyright © 2025 Keyfactor
2+
Copyright 2025 Keyfactor
33
44
Licensed under the Apache License, Version 2.0 (the "License");
55
you may not use this file except in compliance with the License.
@@ -85,12 +85,14 @@ public ICreateCertificateRequestBuilder WithEnrollmentProductInfo(EnrollmentProd
8585
string base64Value = param.Value;
8686

8787
_logger.LogTrace($"Loggin oid and value {oid} {base64Value}");
88-
89-
var extension = CreateX509Extension(oid, base64Value);
90-
if (extension != null)
88+
if (oid != "2.5.29.17") //can't send Sans as an extension to google, they do not like this and you will get an error
9189
{
92-
_logger.LogTrace($"Adding Extension");
93-
_additionalExtensions.Add(extension);
90+
var extension = CreateX509Extension(oid, base64Value);
91+
if (extension != null)
92+
{
93+
_logger.LogTrace($"Adding Extension");
94+
_additionalExtensions.Add(extension);
95+
}
9496
}
9597
}
9698
}

0 commit comments

Comments
 (0)