Changeset 3857

Show
Ignore:
Timestamp:
06/17/08 14:21:53 (2 months ago)
Author:
mgood
Message:

make sure last line of ht files has a newline (fixes #3200)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • accountmanagerplugin/0.10/acct_mgr/htfile.py

    r2171 r3857  
    9393                        print userline 
    9494                    matched = True 
    95                 else: 
     95                elif line.endswith('\n'): 
    9696                    print line, 
     97                else: # make sure the last line has a newline 
     98                    print line 
    9799        except EnvironmentError, e: 
    98100            if e.errno == errno.ENOENT: 

  • accountmanagerplugin/0.10/acct_mgr/tests/htfile.py

    r2171 r3857  
    8686                               'user:$apr1$xW/09...$fb150dT95SoL1HwXtHS/I0') 
    8787 
     88    def test_add_with_no_trailing_newline(self): 
     89        filename = self._create_file('test_add_with_no_trailing_newline', 
     90                                     content='user:$apr1$' 
     91                                             'xW/09...$fb150dT95SoL1HwXtHS/I0') 
     92        self.env.config.set('account-manager', 'password_file', filename) 
     93        self.assertTrue(self.store.check_password('user', 'password')) 
     94        self.store.set_password('user2', 'password2') 
     95        self.assertTrue(self.store.check_password('user', 'password')) 
     96        self.assertTrue(self.store.check_password('user2', 'password2')) 
     97 
    8898 
    8999def suite(): 

  • accountmanagerplugin/trunk/acct_mgr/htfile.py

    r2171 r3857  
    8484                        print userline 
    8585                    matched = True 
    86                 else: 
     86                elif line.endswith('\n'): 
    8787                    print line, 
     88                else: # make sure the last line has a newline 
     89                    print line 
    8890        except EnvironmentError, e: 
    8991            if e.errno == errno.ENOENT: 

  • accountmanagerplugin/trunk/acct_mgr/tests/htfile.py

    r2171 r3857  
    8686                               'user:$apr1$xW/09...$fb150dT95SoL1HwXtHS/I0') 
    8787 
     88    def test_add_with_no_trailing_newline(self): 
     89        filename = self._create_file('test_add_with_no_trailing_newline', 
     90                                     content='user:$apr1$' 
     91                                             'xW/09...$fb150dT95SoL1HwXtHS/I0') 
     92        self.env.config.set('account-manager', 'password_file', filename) 
     93        self.assertTrue(self.store.check_password('user', 'password')) 
     94        self.store.set_password('user2', 'password2') 
     95        self.assertTrue(self.store.check_password('user', 'password')) 
     96        self.assertTrue(self.store.check_password('user2', 'password2')) 
     97 
    8898 
    8999def suite():