fix for username regex

models/site.rb

@@ -683,7 +683,7 @@ class Site < Sequel::Model
   end
 
   def self.valid_username?(username)
-    !username.empty? && username.match(/^[a-zA-Z0-9][a-zA-Z0-9_\-]+[a-zA-Z0-9]$/i)
+    !username.empty? && username.match(/^[a-zA-Z0-9]([a-zA-Z0-9_\-]{0,}[a-zA-Z0-9])?$/i) != nil
   end
 
   def self.disposable_email_domains_whitelist

tests/acceptance/signup_tests.rb

@@ -25,6 +25,10 @@ describe 'signup' do
   end
 
   before do
+    reset_signup
+  end
+
+  def reset_signup
     Capybara.default_driver = :selenium_chrome_headless_largewindow
     Capybara.reset_sessions!
     visit_signup
@@ -107,9 +111,13 @@ describe 'signup' do
     fill_in 'username', with: '|\|0p|E'
     click_signup_button
     _(page).must_have_content 'Usernames can only contain'
+    reset_signup
+    fill_in_valid
     fill_in 'username', with: 'nope-'
     click_signup_button
     _(page).must_have_content 'Usernames can only contain'
+    reset_signup
+    fill_in_valid
     fill_in 'username', with: '-nope'
     click_signup_button
     _(page).must_have_content 'Usernames can only contain'

tests/site_tests.rb

@@ -5,6 +5,17 @@ def app
 end
 
 describe Site do
+  describe 'username' do
+    it 'only passes valid hostnames for username' do
+      _(Site.valid_username?('|\|0p|E')).must_equal false
+      _(Site.valid_username?('nope-')).must_equal false
+      _(Site.valid_username?('-nope')).must_equal false
+      _(Site.valid_username?('do-pe')).must_equal true
+      _(Site.valid_username?('d')).must_equal true
+      _(Site.valid_username?('do')).must_equal true
+    end
+  end
+
   describe 'banning' do
     it 'still makes files available' do
       site = Fabricate :site