Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336631 - Add Proxy Support to file uploads
Summary: Add Proxy Support to file uploads
Status: RESOLVED WONTFIX
Alias: None
Product: EPP
Classification: Technology
Component: Usage Data Collector (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.4.0 M7   Edit
Assignee: Wayne Beaton CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 367402 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-02-08 11:00 EST by Matthew Piggott CLA
Modified: 2021-05-07 10:23 EDT (History)
5 users (show)

See Also:


Attachments
Proxy support for Usage Data Collector (19.85 KB, patch)
2011-02-08 11:00 EST, Matthew Piggott CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Piggott CLA 2011-02-08 11:00:42 EST
Created attachment 188528 [details]
Proxy support for Usage Data Collector

The current implementation of the Usage Data Collector does not support proxies.

The attached patch sets relevant proxies (from org.eclipse.core.net.proxy) for usage data uploads.  To properly set the Content-Length header (required by some proxies) I needed to filter the uploads into temporary files which are also used as the upload source (to avoid duplicate filtering).
Comment 1 Wayne Beaton CLA 2011-02-08 11:44:08 EST
Targeting 1.4.0 M6. Thanks for the contribution.
Comment 2 Matthew Piggott CLA 2011-03-14 15:19:36 EDT
Did you have the opportunity to look at this for M6?
Comment 3 Wayne Beaton CLA 2011-03-17 15:09:33 EDT
I'm trying to grok the changes to BasicUploader. I really liked that FilteredFilePart stuff.

How much of the ProxyHelper comes directly from the ECF implementation? For tracking purposes, it would be helpful if you could separate the contribution into separate patches: one for the ProxyHelper and one for the rest.

Also, can you please assert that you authored these changes and conscent to them being distributed under the terms of the EPL?
Comment 4 Matthew Piggott CLA 2011-03-21 09:51:08 EDT
I authored the changes and consent to them being provided under the EPL.

The difficulty I had with FilteredFilePart was the proxy that I used for testing (Squid) requires an accurate Content-Length header, but when the filtering is performed during the transfer the length of the upload isn't known until after the headers are sent.

In ProxyHelper getProxy, getSocksProxy and selectProxyFromProxies are in the original class, I altered them to remove ECF specific code.  setupProxies is modified from org.eclipse.ecf.provider.filetransfer.httpclient.HttpClientRetrieveFileTransfer

For the two patches, do they need to be able to be applied separately and still compile?
Comment 5 Pascal Rapicault CLA 2011-03-31 14:37:41 EDT
ping.
Comment 6 Pascal Rapicault CLA 2011-05-02 13:26:12 EDT
If you give us commit right, we could apply the patch :)
Comment 7 Markus Knauer CLA 2011-05-20 13:18:04 EDT
ping.
Comment 8 Bron Eubanks CLA 2011-10-04 16:13:41 EDT
I've gotten the same problem with the UDC. Others reported this in Bug 340567 and Bug 241511 (which is closed). However, it might not matter, because according to Bug 347069, the UDC will be removed from Eclipse Juno.
Comment 9 Wayne Beaton CLA 2011-12-22 09:34:04 EST
*** Bug 367402 has been marked as a duplicate of this bug. ***
Comment 10 Jonah Graham CLA 2021-05-07 10:23:04 EDT
The Usage Data Collector was decommissioned a long time ago. Therefore any
remaining bugs are now being closed as WONTFIX. If this bug is still relevant,
please make a comment and we'll move it to the correct project/component for
further investigation.

This change was made as part of a bulk change.