[opensource-dev] Review Request: STORM-1112 Support SOCKS 5 proxy in the viewer (take 2)
Log Linden
log at lindenlab.com
Tue Jul 26 10:20:48 PDT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/374/
-----------------------------------------------------------
(Updated July 26, 2011, 10:20 a.m.)
Review request for Viewer, Oz Linden, Monty Brandenberg, and Stone Linden.
Changes
-------
Added proxy support to other places libcurl is used in the viewer. This is still a WIP, but I wanted to update to match what is available on bitbucket.
Summary
-------
This is a continuation of Robin Cornelius's SOCKS 5 contribution, shown in https://codereview.secondlife.com/r/232/ . I have tried to address all of the comments on that code review and do as much cleanup as possible. The diff includes everything that was submitted by Robin, as well as my work.
Major changes since I started working:
* Changed SOCKS 5 proxy control channel to use the existing LLSocket class, which is a thin wrapper around APR sockets.
* Worked with the Linden Lab UX team to revamp the proxy controls.
* Proxy credentials are now stored in the LLSecAPI password storage, which is the same that is used for users' Second Life Credentials instead of as being stored in the clear as a preference.
This addresses bug STORM-1112.
http://jira.secondlife.com/browse/STORM-1112
Diffs (updated)
-----
indra/llcommon/llsingleton.h 216ebc0442d2
indra/llmessage/CMakeLists.txt 216ebc0442d2
indra/llmessage/llcurl.h 216ebc0442d2
indra/llmessage/llcurl.cpp 216ebc0442d2
indra/llmessage/llhttpassetstorage.cpp 216ebc0442d2
indra/llmessage/llhttpclient.cpp 216ebc0442d2
indra/llmessage/lliosocket.h 216ebc0442d2
indra/llmessage/lliosocket.cpp 216ebc0442d2
indra/llmessage/llpacketring.h 216ebc0442d2
indra/llmessage/llpacketring.cpp 216ebc0442d2
indra/llmessage/llproxy.h PRE-CREATION
indra/llmessage/llproxy.cpp PRE-CREATION
indra/llmessage/llurlrequest.cpp 216ebc0442d2
indra/llmessage/net.h 216ebc0442d2
indra/llmessage/net.cpp 216ebc0442d2
indra/llui/llfunctorregistry.h 216ebc0442d2
indra/newview/app_settings/settings.xml 216ebc0442d2
indra/newview/llappviewer.cpp 216ebc0442d2
indra/newview/llfloaterpreference.h 216ebc0442d2
indra/newview/llfloaterpreference.cpp 216ebc0442d2
indra/newview/llloginhandler.cpp 216ebc0442d2
indra/newview/llpanellogin.h 216ebc0442d2
indra/newview/llsecapi.h 216ebc0442d2
indra/newview/llstartup.h 216ebc0442d2
indra/newview/llstartup.cpp 216ebc0442d2
indra/newview/llviewerfloaterreg.cpp 216ebc0442d2
indra/newview/llxmlrpctransaction.cpp 216ebc0442d2
indra/newview/skins/default/xui/en/floater_preferences_proxy.xml PRE-CREATION
indra/newview/skins/default/xui/en/notifications.xml 216ebc0442d2
indra/newview/skins/default/xui/en/panel_cof_wearables.xml 216ebc0442d2
indra/newview/skins/default/xui/en/panel_preferences_privacy.xml 216ebc0442d2
indra/newview/skins/default/xui/en/panel_preferences_setup.xml 216ebc0442d2
Diff: http://codereview.secondlife.com/r/374/diff
Testing
-------
I've tested exclusively on Linux so far. I'm working on a more extensive test plan that includes setting up a gateway with a restrictive firewall to verify that all traffic is going through the proxy.
Test builds and screenshots of the changed UI elements are available from the project page, located here:
https://wiki.secondlife.com/wiki/User:Log_Linden/Socks5Viewer
Thanks,
Log
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110726/644f2e09/attachment.htm
More information about the opensource-dev
mailing list