Opened 11 years ago

Closed 8 years ago

#8976 closed Patch (outdated)

generic proxy per site

Reported by: Tommy Wu Owned by:
Priority: normal Component: FileZilla Client
Keywords: Cc:
Component version: Operating system type:
Operating system version:

Description

allow to define generic proxy per site.
the rule is:

  1. if define generic proxy per site, it will use it (ignore bypass proxy setting)
  2. if no generic proxy per site, and if bypass proxy, it will connect directly
  3. if no generic proxy per site, and if not bypass proxy, it will use system's generic proxy if defined.

Attachments (5)

filezilla_3.7.3_proxy_per_site.patch​ (29.9 KB ) - added by Tommy Wu 11 years ago.
generic proxy per site
filezilla_3.9.0.6_proxy_per_site.patch​ (28.3 KB ) - added by Tommy Wu 10 years ago.
patch for 3.9.0.6
filezilla_svn6189_proxy_per_site.patch​ (30.0 KB ) - added by Tommy Wu 10 years ago.
fix all issue and update the patch to svn 6189
filezilla_svn6189_proxy_per_site_v2.patch​ (30.0 KB ) - added by Tommy Wu 10 years ago.
fix error for using m_pEngine->GetOptions()
filezilla_svn6189_proxy_per_site_v3.patch​ (30.0 KB ) - added by Tommy Wu 10 years ago.
fix sftp proxy type

Download all attachments as: .zip

Change History (7)

by Tommy Wu, 11 years ago

Attachment: filezilla_3.7.3_proxy_per_site.patch​ added

generic proxy per site

by Tommy Wu, 10 years ago

Attachment: filezilla_3.9.0.6_proxy_per_site.patch​ added

patch for 3.9.0.6

comment:1 by Tim Kosse, 10 years ago

Status: new → moreinfo

Thanks for the update.

Most of the patch already looks good, a couple of comments though:

  1. This patch changes the schema of the queue database. You need to add migration code so that older queues can still be loaded. In CQueueStorage::Impl::MigrateSchema() the new user_version should be 2.
  1. If no proxy is set, store NULLs in the database for the other proxy fields
  1. The SOCKS4 generic proxy type is missing
  1. Indentation seems off in a few places. FileZilla's source is indented with tabs
  1. Please base the patch against SVN trunk

by Tommy Wu, 10 years ago

Attachment: filezilla_svn6189_proxy_per_site.patch​ added

fix all issue and update the patch to svn 6189

by Tommy Wu, 10 years ago

Attachment: filezilla_svn6189_proxy_per_site_v2.patch​ added

fix error for using m_pEngine->GetOptions()

by Tommy Wu, 10 years ago

Attachment: filezilla_svn6189_proxy_per_site_v3.patch​ added

fix sftp proxy type

comment:2 by Kurt McKee, 8 years ago

Resolution: → outdated
Status: moreinfo → closed
Note: See TracTickets for help on using tickets.