[api] backend discards project names with :: - so should the api

openSUSE · Nov 20, 2013 · 775b9c4 · 775b9c4
1 parent 5641467
commit 775b9c4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/api/app/models/project.rb b/src/api/app/models/project.rb
@@ -1123,6 +1123,7 @@ def self.valid_name?(name)
     # this length check is duplicated but useful for other uses for this function
     return false if name.length > 200 || name.blank?
     return false if name =~ %r{^[_\.]} 
+    return false if name =~ %r{::}
     return true if name =~ /\A\w[-+\w\.:]*\z/
     return false
   end

diff --git a/src/api/test/unit/project_test.rb b/src/api/test/unit/project_test.rb
@@ -284,7 +284,10 @@ def put_flags(flags)
     end
   end  
 
-
+  test 'invalid names' do
+    # no ::
+    assert !Project.valid_name?('home:M0ses:raspi::qtdesktop')
+  end
 end