Bug 113712 - Move internet preferences
Summary: Move internet preferences
Status: CLOSED FIXED
Alias: None
Product: WTP Common Tools
Classification: WebTools
Component: wst.internet (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Peter Moogk CLA
QA Contact:
URL:
Whiteboard: 2.0 M1
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-25 17:09 EDT by Chris Brealey CLA
Modified: 2007-05-02 11:21 EDT (History)
1 user (show)

See Also:


Attachments
Patch for plugin org.eclipse.wst.internet.proxy (66.17 KB, patch)
2006-02-13 14:13 EST, Peter Moogk CLA
no flags Details | Diff
New plugin org.eclipse.wst.internet.proxy.ui (23.79 KB, application/octet-stream)
2006-02-13 14:15 EST, Peter Moogk CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Brealey CLA 2005-10-25 17:09:40 EDT
Plugin org.eclipse.wst.internet.proxy contributes a preference page for 
managing internet proxy (firewall) settings. This is a UI plugin which any 
would-be core (non-UI) plugin would have to depend on to easily retrieve or 
update these settings.

A new org.eclipse.wst.internet.proxy.ui plugin should be created as keeper of 
the preference page and associated classes. Any code, utility methods or whole 
classes - namely all of Base64.java and Encoder.java, and bits of 
InternetPlugin.java and InternetPreferencePage.java - that are non-UI in nature 
and assist with the retrieval and storage of the internet proxy preference 
values should be retained in org.eclipse.wst.internet.proxy.

Note that InternetPlugin.java and InternetPreferencePage.java currently use 
AbstractUIPlugin/IPreferenceStore instead of Plugin/Preferences. The code in 
org.eclipse.wst.internet.proxy will have to be changed to use 
Plugin/Preferences to avoid dependencies on the UI.
Comment 1 Chris Brealey CLA 2005-10-25 17:13:44 EDT
I don't think any other plugins in WST currently depend on 
org.eclipse.wst.internet.proxy, so M10 seems a reasonable target to complete 
this change. Also, do not define a public API, since bug 88017 is open to 
migrate the internet proxy facility into the Eclipse platform.
Comment 2 Chris Brealey CLA 2005-11-17 23:46:34 EST
Raising severity to major. Core plugin providers cannot currently get at the 
internet proxy settings in a reliable fashion without depending on the GUI.
Comment 3 Chris Brealey CLA 2005-11-22 11:26:57 EST
Deferring out of WTP 1.0. No other providers of core plugins require access to the internet proxy preferences in WTP 1.0 as far as I know. If this changes, we can reconsider this bug for WTP 1.0.

Peter, assigned to you as the keeper of the internet proxy preferences page.
Comment 4 David Williams CLA 2005-12-28 10:29:27 EST
changing target from 1.5 M1 to 1.5 M5 to reflect new numbering system as we join Collisto. 
Comment 5 Peter Moogk CLA 2006-02-13 14:13:46 EST
Created attachment 34603 [details]
Patch for plugin org.eclipse.wst.internet.proxy
Comment 6 Peter Moogk CLA 2006-02-13 14:15:14 EST
Created attachment 34604 [details]
New plugin org.eclipse.wst.internet.proxy.ui

This zip file contains the files for a new plugin called "org.eclipse.wst.internet.proxy.ui".
Comment 7 Peter Moogk CLA 2006-02-13 14:39:46 EST
This fix involves removing the UI code from the org.eclipse.wst.internet.proxy plugin and creating a new plugin called org.eclipse.wst.internet.proxy.ui.  A new internal api was added to access the proxy JVM system properties.  The class name for this api is org.eclipse.wst.internet.internal.proxy.InternetProperties.  The instance of this new api class can be accessed by calling: 
InternetPlugin.getInstance().getInternetProperties();

Testing for these changes involved checking that the internet preference page still comes up, that the proxy authenticator is still working, and that setting a proxy host and port in the internet preference page still routes internet traffic to a fire wall.  The internet preference page was updated to use this new api which has resulted in this api being tested.
Comment 8 Peter Moogk CLA 2006-02-14 16:01:17 EST
Kathy, this defect is ready to be committed.
Comment 9 Chris Brealey CLA 2006-05-18 15:25:41 EDT
This is not going into WTP 1.5. Will reconsider for WTP 2.0 and/or Eclipse platform.
Comment 10 Chris Brealey CLA 2007-04-17 08:47:04 EDT
Fixed via bug 154100.
Comment 11 Chris Brealey CLA 2007-04-20 15:49:49 EDT
Duplicate.
Comment 12 John Lanuti CLA 2007-05-02 11:21:40 EDT
Closing as verified.