[Fedora-directory-commits] dsgw/tests/domodify testpost.10, NONE, 1.1 testpost.11, NONE, 1.1 testpost.12, NONE, 1.1 testpost.13, NONE, 1.1 testpost.14, NONE, 1.1 testpost.15, NONE, 1.1 testpost.16, NONE, 1.1 testpost.17, NONE, 1.1

[Richard Allen Megginson (rmeggins)]

Author: rmeggins

Update of /cvs/dirsec/dsgw/tests/domodify
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv23811/dsgw/tests/domodify

Added Files:
	testpost.10 testpost.11 testpost.12 testpost.13 testpost.14 
	testpost.15 testpost.16 testpost.17 
Log Message:
1) The old code used a CGI variable called completion_javascript - this variable contained arbitrary javascript code that was eval'd on in the client browser.  I have removed this code and put it in the resource file.  The dsgw code will set completion_javascript to one of the 3 keywords, and the new function emit_completion_javascript will look up the code in the resource file and output it with any required arguments.  It just seems like a really bad idea to execute arbitrary blobs of javascript passed in a CGI argument.

2) Make the checking for the template file names stricter.

3) Added many new tests.

4) When removing unused or duplicate LDAP Mods, if we remove the last one, just free the entire array.



--- NEW FILE testpost.10 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&changed_DN=true&replace_DN_uid=changeduid

--- NEW FILE testpost.11 ---
changetype=modify&dn=uid=changeduid,ou=people,dc=example,dc=com&quiet=&verbose=true&changed_DN=true&replace_DN_uid=scarter

--- NEW FILE testpost.12 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&changed_nosuchattr=true

--- NEW FILE testpost.13 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&delete_nosuchattr=1&changed_nosuchattr=true

--- NEW FILE testpost.14 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&delete_nosuchattr=1&changed_nosuchattr=false

--- NEW FILE testpost.15 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&delete_nosuchattr=1&delete_nosuchattr1=1&changed_nosuchattr=false

--- NEW FILE testpost.16 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&add_ou=
Test OU Foo
Test OU Bar

Test OU Baz
Test OU Biff
&add_mls_description=multiline

$$$$$$$$$$$$$$$$\\\\\\\\\\\\\\\\\\\\\

$$$$$$$$$$$$$$$$\\\\\\\\\\\\\\\\\\\\\

$$$$$$$$$$$$$$$$\\\\\\\\\\\\\\\\\\\\\

$$$$$$$$$$$$$$$$\\\\\\\\\\\\\\\\\\\\\

$$$$$$$$$$$$$$$$\\\\\\\\\\\\\\\\\\\\\

$$$$$$$$$$$$$$$$\\\\\\\\\\\\\\\\\\\\\

description$that$has\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\many\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\lines&add_unique_cn=Test Unique CN

--- NEW FILE testpost.17 ---
changetype=modify&dn=uid=scarter,ou=people,dc=example,dc=com&quiet=&verbose=true&replace_ou=Accounting&replace_ou=People&replace_ou=Test OU&completion_javascript=NOTFOUND