guillotine 0.0.2 → 0.0.3

data/guillotine.gemspec

@@ -13,8 +13,8 @@ Gem::Specification.new do |s|
   ## If your rubyforge_project name is different, then edit it and comment out
   ## the sub! line in the Rakefile
   s.name              = 'guillotine'
-  s.version           = '0.0.2'
-  s.date              = '2011-08-09'
+  s.version           = '0.0.3'
+  s.date              = '2011-08-17'
   s.rubyforge_project = 'guillotine'
 
   ## Make sure your summary is short. The description may be as long

data/lib/guillotine.rb

@@ -2,7 +2,7 @@ require 'base64'
 require 'digest/md5'
 
 module Guillotine
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 
   dir = File.expand_path '../guillotine', __FILE__
   autoload :App, "#{dir}/app"

data/lib/guillotine/app.rb

@@ -12,8 +12,14 @@ module Guillotine
     end
 
     post "/" do
-      url  = params[:url]
+      url  = params[:url].to_s
       code = params[:code]
+
+      if url.empty?
+        halt 422, "Invalid url: #{url.inspect}"
+      end
+      url.strip!
+
       begin
         if code = settings.db.add(url, code)
           redirect code

data/test/app_test.rb

@@ -43,6 +43,16 @@ class AppTest < Guillotine::TestCase
     assert_equal 422, last_response.status
   end
 
+  def test_add_without_url
+    post '/'
+    assert_equal 422, last_response.status
+  end
+
+  def test_add_url_with_linebreak
+    post '/', :url => "https://abc.com\n"
+    assert_equal 'http://example.org/SWtBvQ', last_response.headers['location']
+  end
+
   def app
     Guillotine::App
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: guillotine
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-08-09 00:00:00.000000000Z
+date: 2011-08-17 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sinatra
-  requirement: &70171008407660 !ruby/object:Gem::Requirement
+  requirement: &70155812898660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 1.2.6
   type: :runtime
   prerelease: false
-  version_requirements: *70171008407660
+  version_requirements: *70155812898660
 - !ruby/object:Gem::Dependency
   name: rack-test
-  requirement: &70171008407280 !ruby/object:Gem::Requirement
+  requirement: &70155812898280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +32,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70171008407280
+  version_requirements: *70155812898280
 description: Adaptable private URL shortener
 email: technoweenie@gmail.com
 executables: []