Ashley Davis and GitHub
8b5503ebeb
Merge pull request #4319 from jetstack-bot/cherry-pick-4311-to-release-1.4
...
[release-1.4] Adds an explicit timeout when pre-checking HTTP-01 challenges
v1.4.3
2021-08-06 13:31:38 +01:00
Ashley Davis and jetstack-bot
03ea54d559
adds an explicit timeout when pre-checking HTTP-01 challenges
...
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io >
2021-08-06 11:31:15 +00:00
Ashley Davis and GitHub
8ebc6b665d
Merge pull request #4315 from SgtCoDFish/helmchartk8scompat-1.4
...
[release-1.4] Helm chart k8s compat update
2021-08-06 11:37:51 +01:00
Ashley Davis
1f1530d13f
specify correct version range for cert-manager versions below 1.5
...
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io >
2021-08-05 18:15:10 +01:00
jetstack-bot and GitHub
d0571cb480
Merge pull request #4278 from SgtCoDFish/cherrypick-memleak-1.4
...
[release-1.4] Cherry-pick memory leak fixes into 1.4
v1.4.2
2021-07-29 17:05:41 +01:00
jetstack-bot and GitHub
57cf3fccbd
Merge pull request #4275 from jetstack-bot/cherry-pick-4231-to-release-1.4
...
[release-1.4] Data race: fix concurrent read and write of secret annotations and certificaterequests
2021-07-29 15:56:17 +01:00
Maël Valais and Ashley Davis
a1ad0e943b
memory leak: the afterFunc goroutine now stops properly
...
Signed-off-by: Maël Valais <mael@vls.dev >
2021-07-29 15:46:46 +01:00
Maël Valais and Ashley Davis
6fad252d1f
memory leak: add unit test to show scheduler leaking goroutines
...
Signed-off-by: Maël Valais <mael@vls.dev >
2021-07-29 15:46:30 +01:00
jetstack-bot and GitHub
e288f46a1b
Merge pull request #4272 from jetstack-bot/cherry-pick-4261-to-release-1.4
...
[release-1.4] CA chain fix without root
2021-07-29 15:31:17 +01:00
Maël Valais and jetstack-bot
08196bb28c
data race: fix certificate requests in cache being mutated
...
Signed-off-by: Maël Valais <mael@vls.dev >
2021-07-29 09:15:26 +00:00
Maël Valais and jetstack-bot
41a88bdb1a
data race: fix concurrent read and write of secret annotations
...
This bug can be reproduced using "go run -race" and by creating many
Certificates and renewing them continuously. With 5000 Certificate
objects, a data race is found in less than a minute.
Signed-off-by: Maël Valais <mael@vls.dev >
2021-07-29 09:15:26 +00:00
Ashley Davis and jetstack-bot
d50d4fbd69
handle individual certs in ParseSingleCertificateChain
...
roots are handled differently because they're their own CAs
also adds test cases for each of:
- a lone leaf
- a lone intermediate
- a lone root
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io >
2021-07-28 16:19:16 +00:00
9af0532fd8
Improve ParseSingleCertificateChain when no root is present
...
Fixes when the certificate chain does not have a root CA,
in which case the chain should contain all available intermediates
and ca.crt should contain the rootmost certificate.
Co-authored-by: Josh Van Leeuwen <joshua.vanleeuwen@jetstack.io >
Signed-off-by: Wilson Júnior <wilsonpjunior@gmail.com >
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io >
2021-07-28 16:19:16 +00:00
jetstack-bot and GitHub
f970eca762
Merge pull request #4238 from jetstack-bot/cherry-pick-4237-to-release-1.4
...
[release-1.4] Fix check for self-signed certs in EncodeX509Chain
v1.4.1
2021-07-21 18:35:20 +01:00
Ashley Davis and jetstack-bot
c686a2f69a
fix check for self-signed certs in EncodeX509Chain
...
see also https://github.com/jetstack/cert-manager/issues/4142
EncodeX509Chain checked for self-signed certs by comparing the subject
and issuer of the cert in question, which is invalid since it's
perfectly fine for those to match.
the correct behavior is to use cert.CheckSignatureFrom(cert). this bug
was exposed in 1.4 when ParseSingleCertificateChain started using
EncodeX509Chain in the critical path of several issuers; when end-users
had leaf certificates with subjects matching their issuer's subject, the
bug was triggered.
includes newly written tests for EncodeX509Chain and a test for
ParseSingleCertificateChain
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io >
2021-07-21 16:50:07 +00:00
jetstack-bot and GitHub
655e63122c
Merge pull request #4123 from JoshVanL/release-1.4-e2e-disable-csr-feature-gate
...
Disable running e2e tests for CertificateSigningRequests if feature gate not enabled
2021-06-19 14:26:19 +01:00
joshvanl
160444a827
Disable running e2e tests for CertificateSigningRequests if feature gate
...
is not enabled
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-19 13:43:13 +01:00
jetstack-bot and GitHub
5e2a6883c1
Merge pull request #4092 from irbekrm/renew_override
...
Remove the default renewBefore value
v1.4.0-beta.1
v1.4.0
2021-06-11 16:23:49 +01:00
irbekrm
118cfb6029
Remove the defaulting for renewBefore from fuzzer
...
We now calculate this default at reneal time
Signed-off-by: irbekrm <irbekrm@gmail.com >
2021-06-11 15:35:46 +01:00
jetstack-bot and GitHub
fa40ccdff5
Merge pull request #4090 from JoshVanL/e2e-conformance-vault-check-chain
...
Check certificate chain for Vault Issuer in E2E
2021-06-11 12:29:49 +01:00
irbekrm
acd0a98bbb
Updates DefaultRenewBefore to state that it is deprecated
...
Signed-off-by: irbekrm <irbekrm@gmail.com >
2021-06-11 11:52:54 +01:00
irbekrm
67f14240eb
Update renewBefore descriptions
...
Signed-off-by: irbekrm <irbekrm@gmail.com >
2021-06-11 10:32:36 +01:00
irbekrm
e6b748047d
Remove the default renewBefore value
...
Signed-off-by: irbekrm <irbekrm@gmail.com >
2021-06-11 10:03:12 +01:00
joshvanl
35aaf00395
Check certificate chain for Vault Issuer
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-10 20:25:31 +01:00
jetstack-bot and GitHub
3242b83b12
Merge pull request #4081 from JoshVanL/certificate-signing-request-ca-e2e
...
Certificate Signing Request CA e2e
2021-06-09 13:13:30 +01:00
joshvanl
6dc95cc63c
Adds comment that the kube CSR e2e tests require the feature gate to
...
pass
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-09 11:42:02 +01:00
joshvanl
9ef5fef3a1
Changes kube CSR CA e2e tests to be more readable and improve validation
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-09 11:36:21 +01:00
jetstack-bot and GitHub
4f2f0075a9
Merge pull request #4056 from RinkiyaKeDad/del_framework.go
...
deleting test/e2e/framework/config/framework.go
2021-06-09 11:09:30 +01:00
joshvanl
abdd1f54fa
Fix CA CertificateSigningRequest controller to return potential error
...
from updating failed status
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-07 17:48:49 +01:00
joshvanl
590e01f3d0
Add ExperimentalCertificateSigningRequestControllers=true as default
...
experimental controller to enable in devel/addon/cert-manager/install.md
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-07 17:34:39 +01:00
joshvanl
5a64222475
Adds CA Issuer CertificateSigningRequest e2e test
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-07 15:32:54 +01:00
joshvanl
d4fd4f9acc
Move determining Issuer resource Kind into CSR/util
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-07 15:27:43 +01:00
jetstack-bot and GitHub
5875c828c6
Merge pull request #4074 from JoshVanL/certificate-request-acme-revert-ca.crt
...
Reverts ACME issuer from forming a chain bundle and populating the ca.crt
2021-06-04 16:10:23 +01:00
RinkiyaKeDad
438a0fff13
removed nil line
...
Signed-off-by: RinkiyaKeDad <arshsharma461@gmail.com >
2021-06-04 13:10:04 +05:30
RinkiyaKeDad
662bc5030c
removed more stuff
...
Signed-off-by: RinkiyaKeDad <arshsharma461@gmail.com >
2021-06-03 12:37:27 +05:30
joshvanl
1678d0833e
Reverts ACME issuer from forming a chain bundle and populating the
...
ca.crt
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-06-02 12:21:50 +01:00
jetstack-bot and GitHub
528305b5ed
Merge pull request #4064 from JoshVanL/certificate-request-issuer-ca
...
Certificate Signing Request Issuer CA
v1.4.0-beta.0
2021-05-28 10:57:11 +01:00
joshvanl
36bd7a459c
Changes CSR util signername to use if statements rather than switch
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-28 10:34:43 +01:00
joshvanl
acc5431f1b
Fix signernames to allow clusterissuers with dots in name
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-28 10:13:00 +01:00
joshvanl
9e1b0342d0
Updates with review comments
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 18:48:50 +01:00
jetstack-bot and GitHub
5632379132
Merge pull request #4067 from wallrj/artifacthub-annotations-2
...
Add ArtifactHUB pre-release annotations to the Helm chart
2021-05-27 16:27:14 +01:00
Richard Wall
1f3c3df090
Add ArtifactHUB pre-release annotations to the Helm chart
...
Signed-off-by: Richard Wall <richard.wall@jetstack.io >
2021-05-27 13:58:52 +01:00
jetstack-bot and GitHub
a80198c03d
Merge pull request #4065 from jetstack/revert-4049-artifacthub-annotations
...
Revert "Add ArtifcactHUB pre-release annotation to the Helm chart"
2021-05-27 13:02:34 +01:00
Richard Wall and Richard Wall
8792a17396
Revert "Add ArtifcactHUB pre-release annotation to the Helm chart"
...
Signed-off-by: Richard Wall <richard.wall@jetstack.io >
2021-05-27 12:16:06 +01:00
joshvanl
60d5974115
Moves CertificateSigningRequest controller to feature gate flag
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 12:00:56 +01:00
joshvanl
e014b6655d
Use ca.crt with the CertificateSigningRequest CA controller
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 10:49:21 +01:00
joshvanl
459b5e31b0
RBAC permissions for signing and managing cert-manager
...
CertificateSigningRequersts Issuers
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 00:35:58 +01:00
joshvanl
62dee4783e
Adds CertificateSigningRequest CA Issuer controller as optional
...
controller
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 00:32:24 +01:00
joshvanl
3b74c34089
Adds CertificateSigningRequest CA Issuer controller
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 00:25:02 +01:00
joshvanl
c5c206cace
Adds base CertificateSigningRequest cert-manager controller
...
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com >
2021-05-27 00:23:50 +01:00