psiphon

Homepage - input-validation is broken

Bug #457406 reported by root on 2009-04-14

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	psiphon	Invalid	High	Unassigned

Bug Description

Users can set their default homepage in their profile.
Guest browsing links also have a default homepage.

There are the following problems with input validation when setting a default homepage:
            1. HTTPS addresses are invalid (ie. https://mail.google.com)
            2. Directory segments shorter than two characters are invalid (ie. http://mail.google.com/mail/h)
            3. Trailing slashes are invalid (ie. http://mail.google.com/)

See original description

Tags:

Related branches

lp:~guerrilla/psiphon/sprint3-457334

Ready for review for merging into lp:psiphon

Psiphon Inc.: Pending requested 2010-11-15

Diff: 11766 lines (+11162/-0)

107 files modified

trunk/testing/autotests/ReadMe (+65/-0)
trunk/testing/autotests/ff_profile/bookmarkbackups/bookmarks-2010-09-01.json (+1/-0)
trunk/testing/autotests/ff_profile/cert_override.txt (+4/-0)
trunk/testing/autotests/ff_profile/compatibility.ini (+5/-0)
trunk/testing/autotests/ff_profile/compreg.dat (+1932/-0)
trunk/testing/autotests/ff_profile/extensions.cache (+4/-0)
trunk/testing/autotests/ff_profile/extensions.ini (+6/-0)
trunk/testing/autotests/ff_profile/extensions.rdf (+75/-0)
trunk/testing/autotests/ff_profile/extensions/RCE@jerryqian.com/chrome.manifest (+6/-0)
trunk/testing/autotests/ff_profile/extensions/RCE@jerryqian.com/install.rdf (+30/-0)
trunk/testing/autotests/ff_profile/extensions/RCE@jerryqian.com/license.txt (+13/-0)
trunk/testing/autotests/ff_profile/localstore.rdf (+57/-0)
trunk/testing/autotests/ff_profile/mimeTypes.rdf (+67/-0)
trunk/testing/autotests/ff_profile/pluginreg.dat (+80/-0)
trunk/testing/autotests/ff_profile/prefs.js (+28/-0)
trunk/testing/autotests/ff_profile/search.json (+1/-0)
trunk/testing/autotests/ff_profile/urlclassifierkey3.txt (+2/-0)
trunk/testing/autotests/ff_profile/xpti.dat (+1434/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/LanguageHelper.php (+46/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/NetworkTrafficHelper.php (+14/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/PsiphonTestCase.php (+26/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/PsiphonTestConfig.php (+18/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/Suite.php (+31/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/Testing/Selenium/Exception/Exception.php (+40/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/Testing/Selenium/ModifiedSelenium.php (+65/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/Testing/Selenium/Selenium.php (+2755/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/UserActions.php (+61/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/bootstrap.php (+16/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/configuration.xml (+26/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon10TestCase.php (+44/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon11TestCase.php (+52/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon12TestCase.php (+82/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon13TestCase.php (+37/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon14TestCase.php (+47/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon16TestCase.php (+37/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon17TestCase.php (+46/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon18TestCase.php (+35/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon19TestCase.php (+30/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon1TestCase.php (+42/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon20TestCase.php (+27/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon21TestCase.php (+35/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon22TestCase.php (+120/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon23TestCase.php (+49/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon24TestCase.php (+43/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon25TestCase.php (+35/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon26TestCase.php (+55/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon27TestCase.php (+63/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon28TestCase.php (+93/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon29TestCase.php (+42/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon2TestCase.php (+41/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon30TestCase.php (+38/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon31TestCase.php (+110/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon32TestCase.php (+49/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon33TestCase.php (+61/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon34TestCase.php (+64/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon35TestCase.php (+34/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon36TestCase.php (+119/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon37TestCase.php (+48/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon38TestCase.php (+311/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon39TestCase.php (+73/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon3TestCase.php (+43/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon40TestCase.php (+80/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon41TestCase.php (+58/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon43TestCase.php (+60/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon44TestCase.php (+61/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon45TestCase.php (+66/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon46TestCase.php (+71/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon47TestCase.php (+54/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon48TestCase.php (+63/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon49TestCase.php (+37/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon4TestCase.php (+37/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon50TestCase.php (+23/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon51TestCase.php (+28/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon52TestCase.php (+79/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon53TestCase.php (+113/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon54TestCase.php (+64/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon55TestCase.php (+55/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon56TestCase.php (+66/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon57TestCase.php (+66/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon58TestCase.php (+75/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon59TestCase.php (+60/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon5TestCase.php (+48/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon60TestCase.php (+41/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon61TestCase.php (+28/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon62TestCase.php (+121/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon63TestCase.php (+38/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon64TestCase.php (+40/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon65TestCase.php (+40/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon66TestCase.php (+58/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon67TestCase.php (+70/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon68TestCase.php (+30/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon69TestCase.php (+22/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon6TestCase.php (+45/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon70TestCase.php (+33/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon71TestCase.php (+72/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon72TestCase.php (+66/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon74TestCase.php (+57/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon75TestCase.php (+69/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon7TestCase.php (+12/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon8TestCase.php (+54/-0)
trunk/testing/autotests/psiphoneAutoTests/Test Files/tests/Psiphon9TestCase.php (+53/-0)
trunk/testing/autotests/psiphoneAutoTests/index.php (+6/-0)
trunk/testing/autotests/psiphoneAutoTests/nbproject/private/private.properties (+5/-0)
trunk/testing/autotests/psiphoneAutoTests/nbproject/private/private.xml (+4/-0)
trunk/testing/autotests/psiphoneAutoTests/nbproject/project.properties (+11/-0)
trunk/testing/autotests/psiphoneAutoTests/nbproject/project.xml (+9/-0)
trunk/testing/autotests/selenium_start (+1/-0)

Revision history for this message

root (n-root-psiphon-ca) wrote on 2009-06-18:

Milestone May 2009 Release deleted

Adam P (adam+) on 2009-10-29

Changed in psiphon:
status:	New → Confirmed

Chris (poser) on 2009-10-30

tags:

added: poser

Chris (poser) on 2009-10-30

tags:

removed: poser

Rod (rod-psiphon) on 2009-11-24

visibility:

private → public

Rod (rod-psiphon) on 2010-05-06

tags:

added: category2

Adam P (adam+) on 2010-07-09

tags:

added: category2.1

Adam Kruger (adam-kruger) on 2010-07-27

description:	updated
description:	updated
Changed in psiphon:
importance:	Unknown → High

Adam Kruger (adam-kruger) on 2010-07-27

tags:

added: sprint3

Revision history for this message

Nurlan Turdaliev (nurlan0000) wrote on 2010-08-31:

is_url Funciton test Edit (469 bytes, application/x-httpd-php)

Cannot reproduce, see attachment script.

Revision history for this message

Adam Kruger (adam-kruger) wrote on 2010-09-07:

Please ensure that an automated test case for this bug is added to the full test suite.

Revision history for this message

Robert Vasilev (vasilev) wrote on 2010-09-14:

Automated test which covers this issue has been added to Test Case #71.

Currently at:
http://bazaar.launchpad.net/~guerrilla/psiphon/sprint3-457334/revision/127/trunk/testing/autotests/psiphoneAutoTests/Test%20Files/tests/Psiphon71TestCase.php

Changed in psiphon:
status:	Confirmed → Invalid
status:	Invalid → Confirmed

Adam Kruger (adam-kruger) on 2010-10-20

Changed in psiphon:
status:	Confirmed → Invalid

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Bug attachments

is_url Funciton test Edit

Add attachment

Remote bug watches

Bug watches keep track of this bug in other bug trackers.