Skip to content

Commit

Permalink
Ask user to create new endpoint, show ip price
Browse files Browse the repository at this point in the history 
[#85083432]
  • Loading branch information
@sabcio
sabcio committed Jan 2, 2015
1 parent f6a14da commit 700c28c
Show file tree
Hide file tree
Showing 2 changed files with 122 additions and 31 deletions.
23 changes: 22 additions & 1 deletion lib/shelly/cli/endpoint.rb
View file
Expand Up @@ -34,7 +34,6 @@ def list
end

print_table(to_display, :ident => 2)

else
say "No HTTP endpoints available"
end
Expand Down Expand Up @@ -81,6 +80,18 @@ def create(cert_path = nil, key_path = nil, bundle_path = nil)
app = multiple_clouds(options[:cloud],
"endpoint create [CERT_PATH] [KEY_PATH] [BUNDLE_PATH]")

say "Every unique IP address assigned to endpoint costs 10€/month"
say "It's required for SSL/TLS"
if cert_path == nil && key_path == nil
say "You didn't provide certificate but it can be added later"
say "Assigned IP address can be used to catch all domains pointing to that address, without SSL/TLS enabled"
exit(0) unless yes?("Are you sure you want to create endpoint without certificate (yes/no):")
elsif app.endpoints.count > 0
ask_if_endpoints_were_already_created(app)
else
exit(0) unless yes?("Are you sure you want to create endpoint? (yes/no):")
end

certificate, key = read_certificate_components(cert_path, key_path,
bundle_path)

Expand Down Expand Up @@ -172,6 +183,16 @@ def read_certificate_components(cert_path, key_path, bundle_path)
[certificate, key]
end
end

def ask_if_endpoints_were_already_created(app)
cli = Shelly::CLI::Endpoint.new
cli.options = {:cloud => app}
cli.list
say_new_line
exit(0) unless yes?("You already have assigned endpoint(s). Are " \
"you sure you want to create another one with a new IP address? " \
"(yes/no):")
end
end
end
end
Expand Down
130 changes: 100 additions & 30 deletions spec/shelly/cli/endpoint_spec.rb
View file
Expand Up @@ -29,31 +29,6 @@
invoke(@cli, :list)
end

def endpoints_response
[
{'ip_address' => '10.0.0.1',
'sni' => true,
'uuid' => 'uuid1', 'info' => {
'domain' => 'example.com',
'issuer' => 'CA',
'subjcet' => 'organization info',
'since' => '2012-06-11 23:00:00 UTC',
'to' => '2015-06-11 23:00:00 UTC'
}
},
{'ip_address' => '10.0.0.2',
'sni' => false,
'uuid' => 'uuid2', 'info' => {
'domain' => nil,
'issuer' => nil,
'subjcet' => nil,
'since' => nil,
'to' => nil
}
}
]
end

context "no endpoints" do
it "should display information" do
@app.should_receive(:endpoints).and_return([])
Expand Down Expand Up @@ -98,39 +73,103 @@ def endpoints_response
File.stub(:read).with('crt_path').and_return('crt')
File.stub(:read).with('key_path').and_return('key')
File.stub(:read).with('bundle_path').and_return('bundle')
@app.stub(:endpoints).and_return([])
end

it "should create endpoint" do
it "should create endpoint with provided certificate" do
@app.should_receive(:create_endpoint).with("crt\n", "key", true).
and_return(endpoint_response('ip_address' => '10.0.0.1'))

$stdout.should_receive(:puts).with("Every unique IP address assigned to endpoint costs 10\u20AC/month")
$stdout.should_receive(:puts).with("It's required for SSL/TLS")
$stdout.should_receive(:print).with("Are you sure you want to create endpoint? (yes/no): ")

$stdout.should_receive(:puts).with(green "Endpoint was created for #{@app.to_s} cloud")
$stdout.should_receive(:puts).with("Deployed certificate on front end servers.")
$stdout.should_receive(:puts).with("Point your domain to private IP address: 10.0.0.1")

@cli.options = {"sni" => true}
invoke(@cli, :create, "crt_path", "key_path")
fake_stdin(["yes"]) do
invoke(@cli, :create, "crt_path", "key_path")
end
end

context "without certificate" do
it "should create endpoint" do
@app.should_receive(:create_endpoint).with(nil, nil, true).
and_return(endpoint_response('ip_address' => '10.0.0.1'))

$stdout.should_receive(:puts).with("Every unique IP address assigned to endpoint costs 10\u20AC/month")
$stdout.should_receive(:puts).with("It's required for SSL/TLS")
$stdout.should_receive(:puts).with("You didn't provide certificate but it can be added later")
$stdout.should_receive(:puts).with("Assigned IP address can be used to catch all domains pointing to that address, without SSL/TLS enabled")
$stdout.should_receive(:print).with("Are you sure you want to create endpoint without certificate (yes/no): ")
$stdout.should_receive(:puts).with(green "Endpoint was created for #{@app.to_s} cloud")
$stdout.should_receive(:puts).with("Point your domain to private IP address: 10.0.0.1")

@cli.options = {"sni" => true}
fake_stdin(["yes"]) do
invoke(@cli, :create)
end
end
end

context "multiple endpoints" do
it "should create endpoint with provided certificate" do
@app.should_receive(:create_endpoint).with("crt\n", "key", nil).
and_return(endpoint_response('ip_address' => '10.0.0.1'))
@app.stub(:endpoints).and_return(endpoints_response)

$stdout.should_receive(:puts).with("Every unique IP address assigned to endpoint costs 10\u20AC/month")
$stdout.should_receive(:puts).with("It's required for SSL/TLS")
$stdout.should_receive(:puts).with(green "Available HTTP endpoints")
$stdout.should_receive(:puts).with("\n")
$stdout.should_receive(:puts).with(" UUID | IP address | Certificate | SNI")
$stdout.should_receive(:puts).with(" uuid1 | 10.0.0.1 | example.com | \u2713")
$stdout.should_receive(:puts).with(" uuid2 | 10.0.0.2 | \u2717 | \u2717")
$stdout.should_receive(:puts).with("\n")
$stdout.should_receive(:print).with("You already have assigned endpoint(s). Are you sure you want to create another one with a new IP address? (yes/no): ")
$stdout.should_receive(:puts).with(green "Endpoint was created for #{@app.to_s} cloud")
$stdout.should_receive(:puts).with("Deployed certificate on front end servers.")
$stdout.should_receive(:puts).with("Point your domain to private IP address: 10.0.0.1")

fake_stdin(["yes"]) do
invoke(@cli, :create, "crt_path", "key_path")
end
end
end

context "validation errors" do
it "should show errors and exit" do
exception = Shelly::Client::ValidationException.new({"errors" => [["key", "is invalid"]]})
@app.should_receive(:create_endpoint).and_raise(exception)

$stdout.should_receive(:puts).with("Every unique IP address assigned to endpoint costs 10\u20AC/month")
$stdout.should_receive(:puts).with("It's required for SSL/TLS")
$stdout.should_receive(:print).with("Are you sure you want to create endpoint? (yes/no): ")
$stdout.should_receive(:puts).with(red "Key is invalid")

lambda {
invoke(@cli, :create, "crt_path", "key_path", "bundle_path")
fake_stdin(["yes"]) do
invoke(@cli, :create, "crt_path", "key_path", "bundle_path")
end
}.should raise_error(SystemExit)
end
end

context "providing one only part of certificate" do
it "should show error and exit" do
@app.should_not_receive(:create_endpoint)

$stdout.should_receive(:puts).with("Every unique IP address assigned to endpoint costs 10\u20AC/month")
$stdout.should_receive(:puts).with("It's required for SSL/TLS")
$stdout.should_receive(:print).with("Are you sure you want to create endpoint? (yes/no): ")
$stdout.should_receive(:puts).with(red "Provide both certificate and key")

lambda {
invoke(@cli, :create, "crt_path")
fake_stdin(["yes"]) do
invoke(@cli, :create, "crt_path")
end
}.should raise_error(SystemExit)
end
end
Expand All @@ -140,10 +179,16 @@ def endpoints_response
exception = Shelly::Client::ConflictException.new("message" =>
"That's an error")
@app.should_receive(:create_endpoint).and_raise(exception)

$stdout.should_receive(:puts).with("Every unique IP address assigned to endpoint costs 10\u20AC/month")
$stdout.should_receive(:puts).with("It's required for SSL/TLS")
$stdout.should_receive(:print).with("Are you sure you want to create endpoint? (yes/no): ")
$stdout.should_receive(:puts).with(red "That's an error")

lambda {
invoke(@cli, :create, "crt_path", "key_path", "bundle_path")
fake_stdin(["yes"]) do
invoke(@cli, :create, "crt_path", "key_path", "bundle_path")
end
}.should raise_error(SystemExit)
end
end
Expand Down Expand Up @@ -209,6 +254,31 @@ def endpoints_response
end
end

def endpoints_response
[
{ 'ip_address' => '10.0.0.1',
'sni' => true,
'uuid' => 'uuid1', 'info' => {
'domain' => 'example.com',
'issuer' => 'CA',
'subjcet' => 'organization info',
'since' => '2012-06-11 23:00:00 UTC',
'to' => '2015-06-11 23:00:00 UTC'
}
},
{ 'ip_address' => '10.0.0.2',
'sni' => false,
'uuid' => 'uuid2', 'info' => {
'domain' => nil,
'issuer' => nil,
'subjcet' => nil,
'since' => nil,
'to' => nil
}
}
]
end

def endpoint_response(options = {})
{'ip_address' => '10.0.0.1', 'sni' => true,
'uuid' => 'uuid1', 'info' => {
Expand Down

0 comments on commit 700c28c

Please sign in to comment.