1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
|
#! /usr/bin/env ruby
require 'spec_helper'
require 'puppet/face'
describe Puppet::Face[:ca, '0.1.0'], :unless => Puppet.features.microsoft_windows? do
include PuppetSpec::Files
before :each do
Puppet.run_mode.stubs(:master?).returns(true)
Puppet[:ca] = true
Puppet[:ssldir] = tmpdir("face-ca-ssldir")
Puppet::SSL::Host.ca_location = :only
Puppet[:certificate_revocation] = true
# This is way more intimate than I want to be with the implementation, but
# there doesn't seem any other way to test this. --daniel 2011-07-18
Puppet::SSL::CertificateAuthority.stubs(:instance).returns(
# ...and this actually does the directory creation, etc.
Puppet::SSL::CertificateAuthority.new
)
end
def given_certificate_requests_for(*names)
names.each do |name|
Puppet::SSL::Host.new(name).generate_certificate_request
end
end
def given_certificates_for(*names)
names.each do |name|
Puppet::SSL::Host.new(name).generate
end
end
context "#verify" do
it "should report if there is no certificate" do
subject.verify('random-host').should == {
:host => 'random-host', :valid => false,
:error => 'Could not find a certificate for random-host'
}
end
it "should report that it cannot find a certificate when there is only a request" do
given_certificate_requests_for('random-host')
subject.verify('random-host').should == {
:host => 'random-host', :valid => false,
:error => 'Could not find a certificate for random-host'
}
end
it "should verify a signed certificate" do
given_certificates_for('random-host')
subject.verify('random-host').should == {
:host => 'random-host', :valid => true
}
end
it "should not verify a revoked certificate" do
given_certificates_for('random-host')
subject.revoke('random-host')
subject.verify('random-host').should == {
:host => 'random-host', :valid => false,
:error => 'certificate revoked'
}
end
it "should verify a revoked certificate if CRL use was turned off" do
given_certificates_for('random-host')
subject.revoke('random-host')
Puppet[:certificate_revocation] = false
subject.verify('random-host').should == {
:host => 'random-host', :valid => true
}
end
end
context "#fingerprint" do
let(:fingerprint_re) { /^\([0-9A-Z]+\) [0-9A-F:]+$/ }
it "should be nil if there is no certificate" do
subject.fingerprint('random-host').should be_nil
end
it "should fingerprint a CSR" do
given_certificate_requests_for('random-host')
subject.fingerprint('random-host').should =~ fingerprint_re
end
it "should fingerprint a certificate" do
given_certificates_for('random-host')
subject.fingerprint('random-host').should =~ fingerprint_re
end
%w{md5 MD5 sha1 SHA1 RIPEMD160 sha256 sha512}.each do |digest|
it "should fingerprint with #{digest.inspect}" do
given_certificates_for('random-host')
subject.fingerprint('random-host', :digest => digest).should =~ fingerprint_re
end
end
end
context "#print" do
it "should be nil if there is no certificate" do
subject.print('random-host').should be_nil
end
it "should return nothing if there is only a CSR" do
given_certificate_requests_for('random-host')
subject.print('random-host').should be_nil
end
it "should return the certificate content if there is a cert" do
given_certificates_for('random-host')
text = subject.print('random-host')
text.should be_an_instance_of String
text.should =~ /^Certificate:/
text.should =~ /Issuer: CN=Puppet CA: /
text.should =~ /Subject: CN=random-host$/
end
end
context "#sign" do
it "should report that there is no CSR" do
subject.sign('random-host').should == 'Could not find certificate request for random-host'
end
it "should report that there is no CSR when even when there is a certificate" do
given_certificates_for('random-host')
subject.sign('random-host').should == 'Could not find certificate request for random-host'
end
it "should sign a CSR if one exists" do
given_certificate_requests_for('random-host')
subject.sign('random-host').should be_an_instance_of Puppet::SSL::Certificate
list = subject.list(:signed => true)
list.length.should == 1
list.first.name.should == 'random-host'
end
describe "when the CSR specifies DNS alt names" do
let(:host) { Puppet::SSL::Host.new('someone') }
before :each do
host.generate_certificate_request(:dns_alt_names => 'some,alt,names')
end
it "should sign the CSR if DNS alt names are allowed" do
subject.sign('someone', :allow_dns_alt_names => true)
host.certificate.should be_a(Puppet::SSL::Certificate)
end
it "should refuse to sign the CSR if DNS alt names are not allowed" do
certname = 'someone'
expect do
subject.sign(certname)
end.to raise_error(Puppet::SSL::CertificateAuthority::CertificateSigningError, /CSR '#{certname}' contains subject alternative names \(.*\), which are disallowed. Use `puppet cert --allow-dns-alt-names sign #{certname}` to sign this request./i)
host.certificate.should be_nil
end
end
end
context "#generate" do
it "should generate a certificate if requested" do
subject.list(:all => true).should == []
subject.generate('random-host')
list = subject.list(:signed => true)
list.length.should == 1
list.first.name.should == 'random-host'
end
it "should report if a CSR with that name already exists" do
given_certificate_requests_for('random-host')
subject.generate('random-host').should =~ /already has a certificate request/
end
it "should report if the certificate with that name already exists" do
given_certificates_for('random-host')
subject.generate('random-host').should =~ /already has a certificate/
end
it "should include the specified DNS alt names" do
subject.generate('some-host', :dns_alt_names => 'some,alt,names')
host = subject.list(:signed => true).first
host.name.should == 'some-host'
host.certificate.subject_alt_names.should =~ %w[DNS:some DNS:alt DNS:names DNS:some-host]
subject.list(:pending => true).should be_empty
end
end
context "#revoke" do
it "should let the user know what went wrong when there is nothing to revoke" do
subject.revoke('nonesuch').should == 'Nothing was revoked'
end
it "should revoke a certificate" do
given_certificates_for('random-host')
subject.revoke('random-host')
found = subject.list(:all => true, :subject => 'random-host')
subject.get_action(:list).when_rendering(:console).call(found, {}).
should =~ /^- random-host \(\w+\) [:0-9A-F]+ \(certificate revoked\)/
end
end
context "#destroy" do
it "should let the user know if nothing was deleted" do
subject.destroy('nonesuch').should == "Nothing was deleted"
end
it "should destroy a CSR, if we have one" do
given_certificate_requests_for('random-host')
subject.destroy('random-host')
subject.list(:pending => true, :subject => 'random-host').should == []
end
it "should destroy a certificate, if we have one" do
given_certificates_for('random-host')
subject.destroy('random-host')
subject.list(:signed => true, :subject => 'random-host').should == []
end
it "should tell the user something was deleted" do
given_certificates_for('random-host')
subject.list(:signed => true, :subject => 'random-host').should_not == []
subject.destroy('random-host').
should == "Deleted for random-host: Puppet::SSL::Certificate"
end
end
context "#list" do
context "with no hosts in CA" do
[
{},
{ :pending => true },
{ :signed => true },
{ :all => true },
].each do |type|
it "should return nothing for #{type.inspect}" do
subject.list(type).should == []
end
it "should return nothing when a matcher is passed" do
subject.list(type.merge :subject => '.').should == []
end
context "when_rendering :console" do
it "should return nothing for #{type.inspect}" do
subject.get_action(:list).when_rendering(:console).call(subject.list(type), {}).should == ""
end
end
end
end
context "with some hosts" do
csr_names = (1..3).map {|n| "csr-#{n}" }
crt_names = (1..3).map {|n| "crt-#{n}" }
all_names = csr_names + crt_names
{
{} => csr_names,
{ :pending => true } => csr_names,
{ :signed => true } => crt_names,
{ :all => true } => all_names,
{ :pending => true, :signed => true } => all_names,
}.each do |input, expect|
it "should map #{input.inspect} to #{expect.inspect}" do
given_certificate_requests_for(*csr_names)
given_certificates_for(*crt_names)
subject.list(input).map(&:name).should =~ expect
end
['', '.', '2', 'none'].each do |pattern|
filtered = expect.select {|x| Regexp.new(pattern).match(x) }
it "should filter all hosts matching #{pattern.inspect} to #{filtered.inspect}" do
given_certificate_requests_for(*csr_names)
given_certificates_for(*crt_names)
subject.list(input.merge :subject => pattern).map(&:name).should =~ filtered
end
end
end
context "when_rendering :console" do
{ [["csr1.local"], []] => [/^ csr1.local /],
[[], ["crt1.local"]] => [/^\+ crt1.local /],
[["csr2"], ["crt2"]] => [/^ csr2 /, /^\+ crt2 /]
}.each do |input, pattern|
it "should render #{input.inspect} to match #{pattern.inspect}" do
given_certificate_requests_for(*input[0])
given_certificates_for(*input[1])
text = subject.get_action(:list).when_rendering(:console).call(subject.list(:all => true), {})
pattern.each do |item|
text.should =~ item
end
end
end
end
end
end
actions = %w{destroy list revoke generate sign print verify fingerprint}
actions.each do |action|
it { should be_action action }
it "should fail #{action} when not a CA" do
Puppet[:ca] = false
expect {
case subject.method(action).arity
when -1 then subject.send(action)
when -2 then subject.send(action, 'dummy')
else
raise "#{action} has arity #{subject.method(action).arity}"
end
}.to raise_error(/Not a CA/)
end
end
end
|