<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>56499</bug_id>
          
          <creation_ts>2011-03-16 16:14:42 -0700</creation_ts>
          <short_desc>webkit-patch does not support HTTPS proxies due to old version of Mechanize</short_desc>
          <delta_ts>2011-03-17 05:11:35 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Keith Kyzivat">kamaji</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>benjamin</cc>
    
    <cc>cjerdonek</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>kling</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>levin</cc>
    
    <cc>tony</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>368615</commentid>
    <comment_count>0</comment_count>
    <who name="Keith Kyzivat">kamaji</who>
    <bug_when>2011-03-16 16:14:42 -0700</bug_when>
    <thetext>webkit-patch currently does not support HTTPS proxies.  Many companies use proxies to access the public internet, and this is needed in order to be able to use webkit-patch.

This is due to a limitation in the older version of mechanize that we use.
Mechanize supports HTTPS proxies in newer versions of Mechanize, and I have determined that version 0.2.4 works fine with webkit-patch upload (I have not tested more than that).

Here is a post from the author of mechanize indicating that HTTPS proxy support was present in his git repo as of 31-Jan-2010, and it&apos;s been a year since then, and 5 releases since 0.1.11.
http://blog.gmane.org/gmane.comp.python.wwwsearch.general/month=20100201
http://pypi.python.org/packages/source/m/mechanize/

I would like to see the webkit scripts update to use mechanize 0.2.4 if possible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>368622</commentid>
    <comment_count>1</comment_count>
      <attachid>85996</attachid>
    <who name="Keith Kyzivat">kamaji</who>
    <bug_when>2011-03-16 16:24:21 -0700</bug_when>
    <thetext>Created attachment 85996
Patch

update mechanize to 0.2.4</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>368782</commentid>
    <comment_count>2</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2011-03-16 20:16:23 -0700</bug_when>
    <thetext>CC&apos;ing folks who have poked the autoinstall stuff in the past.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>368833</commentid>
    <comment_count>3</comment_count>
      <attachid>85996</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-03-16 22:48:11 -0700</bug_when>
    <thetext>Comment on attachment 85996
Patch

Looks fine, but Eric should take a look too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>368835</commentid>
    <comment_count>4</comment_count>
      <attachid>85996</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2011-03-16 22:48:45 -0700</bug_when>
    <thetext>Comment on attachment 85996
Patch

LGTM!  Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>369011</commentid>
    <comment_count>5</comment_count>
      <attachid>85996</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-03-17 05:11:29 -0700</bug_when>
    <thetext>Comment on attachment 85996
Patch

Clearing flags on attachment: 85996

Committed r81348: &lt;http://trac.webkit.org/changeset/81348&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>369012</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-03-17 05:11:35 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>85996</attachid>
            <date>2011-03-16 16:24:21 -0700</date>
            <delta_ts>2011-03-17 05:11:29 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-56499-20110316192519.patch</filename>
            <type>text/plain</type>
            <size>3005</size>
            <attacher name="Keith Kyzivat">kamaji</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODEyODAKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBi
L1Rvb2xzL0NoYW5nZUxvZwppbmRleCAzNDY1NWMxYzFkODI5NmUxM2EwNjViYmJjZTk0ODdhYzgw
NjhjOTY2Li4yYmZkZmU4MjFiOGQ4MmVkMjdhNGIyYjJkOWZmNWI3Njg2NzY5ZDRiIDEwMDY0NAot
LS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYg
QEAKKzIwMTEtMDMtMTYgIEtlaXRoIEt5eml2YXQgIDxrZWl0aC5reXppdmF0QG5va2lhLmNvbT4K
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBVcGRhdGUg
YXV0b2luc3RhbGwgdG8gaW5zdGFsbCBtZWNoYW5pemUgMC4yLjQKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTU2NDk5CisKKyAgICAgICAgTmV3ZXIgdmVy
c2lvbnMgb2YgbWVjaGFuaXplIHN1cHBvcnQgSFRUUFMgcHJveGllcywgd2hpY2ggaXMgbmVjZXNz
YXJ5CisgICAgICAgIGZvciBwZW9wbGUgdXNpbmcgd2Via2l0LXBhdGNoIGJlaGluZCBmaXJld2Fs
bHMuCisKKyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9zeXN0ZW0vYXV0b2luc3Rh
bGwucHk6CisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS90aGlyZHBhcnR5L19faW5pdF9fLnB5
OgorCiAyMDExLTAzLTE1ICBIYXlhdG8gSXRvICA8aGF5YXRvQGNocm9taXVtLm9yZz4KIAogICAg
ICAgICBSZXZpZXdlZCBieSBPamFuIFZhZmFpLgpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy93
ZWJraXRweS9jb21tb24vc3lzdGVtL2F1dG9pbnN0YWxsLnB5IGIvVG9vbHMvU2NyaXB0cy93ZWJr
aXRweS9jb21tb24vc3lzdGVtL2F1dG9pbnN0YWxsLnB5CmluZGV4IDlhZGFiMjkwYzE0YWU1ZGU4
NjE5OGVmNjgwMjg1YmU2Y2Q0NDQ5MGEuLjRmZmNjY2M2NDY1NTc2M2NiN2RjM2M3ZGI2ZjM3NGNk
NGJlMjQ3YjEgMTAwNzU1Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL3N5c3Rl
bS9hdXRvaW5zdGFsbC5weQorKysgYi9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9zeXN0
ZW0vYXV0b2luc3RhbGwucHkKQEAgLTYxLDcgKzYxLDcgQEAgY2xhc3MgQXV0b0luc3RhbGxlcihv
YmplY3QpOgogCiAgICAgaW5zdGFsbGVyLmluc3RhbGwodXJsPSJodHRwOi8vcHlwaS5weXRob24u
b3JnL3BhY2thZ2VzL3NvdXJjZS9wL3BlcDgvcGVwOC0wLjUuMC50YXIuZ3ojbWQ1PTUxMmE4MThh
Zjk5NzkyOTBjZDYxOWNjZThlOWMyZTJiIiwKICAgICAgICAgICAgICAgICAgICAgICB1cmxfc3Vi
cGF0aD0icGVwOC0wLjUuMC9wZXA4LnB5IikKLSAgICBpbnN0YWxsZXIuaW5zdGFsbCh1cmw9Imh0
dHA6Ly9weXBpLnB5dGhvbi5vcmcvcGFja2FnZXMvc291cmNlL20vbWVjaGFuaXplL21lY2hhbml6
ZS0wLjEuMTEuemlwIiwKKyAgICBpbnN0YWxsZXIuaW5zdGFsbCh1cmw9Imh0dHA6Ly9weXBpLnB5
dGhvbi5vcmcvcGFja2FnZXMvc291cmNlL20vbWVjaGFuaXplL21lY2hhbml6ZS0wLjIuNC56aXAi
LAogICAgICAgICAgICAgICAgICAgICAgIHVybF9zdWJwYXRoPSJtZWNoYW5pemUiKQogCiAgICAg
IiIiCkBAIC01MTIsNiArNTEyLDYgQEAgaWYgX19uYW1lX189PSJfX21haW5fXyI6CiAgICAgICAg
ICAgICAgICAgICAgICAgdXJsX3N1YnBhdGg9InBlcDgtMC41LjAvcGVwOC5weSIpCiAgICAgaW5z
dGFsbGVyLmluc3RhbGwoc2hvdWxkX3JlZnJlc2g9RmFsc2UsCiAgICAgICAgICAgICAgICAgICAg
ICAgdGFyZ2V0X25hbWU9Im1lY2hhbml6ZSIsCi0gICAgICAgICAgICAgICAgICAgICAgdXJsPSJo
dHRwOi8vcHlwaS5weXRob24ub3JnL3BhY2thZ2VzL3NvdXJjZS9tL21lY2hhbml6ZS9tZWNoYW5p
emUtMC4xLjExLnppcCIsCisgICAgICAgICAgICAgICAgICAgICAgdXJsPSJodHRwOi8vcHlwaS5w
eXRob24ub3JnL3BhY2thZ2VzL3NvdXJjZS9tL21lY2hhbml6ZS9tZWNoYW5pemUtMC4yLjQuemlw
IiwKICAgICAgICAgICAgICAgICAgICAgICB1cmxfc3VicGF0aD0ibWVjaGFuaXplIikKIApkaWZm
IC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy93ZWJraXRweS90aGlyZHBhcnR5L19faW5pdF9fLnB5IGIv
VG9vbHMvU2NyaXB0cy93ZWJraXRweS90aGlyZHBhcnR5L19faW5pdF9fLnB5CmluZGV4IGMyMjQ5
YzJmNmZhMWNmMmRlNDM2NTAzODYzZmI0YzMzOGU1OWU2NWIuLjk3Mjg0OTI1MGI3YmY1ZTg0NjQ1
MzJiZDAwYjBkZWRmMzQ5ZTU2YzcgMTAwNjQ0Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkv
dGhpcmRwYXJ0eS9fX2luaXRfXy5weQorKysgYi9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L3RoaXJk
cGFydHkvX19pbml0X18ucHkKQEAgLTcwLDcgKzcwLDcgQEAgaW5zdGFsbGVyLmluc3RhbGwodXJs
PSJodHRwOi8vcHlwaS5weXRob24ub3JnL3BhY2thZ2VzL3NvdXJjZS9DL0NsaWVudEZvcm0vQ2xp
ZW4KICMgYSBuZXcgQXV0b0luc3RhbGxlciBpbnN0YW5jZSB0aGF0IGRvZXMgbm90IGFwcGVuZCB0
byB0aGUgc2VhcmNoIHBhdGguCiBpbnN0YWxsZXIgPSBBdXRvSW5zdGFsbGVyKHRhcmdldF9kaXI9
YXV0b2luc3RhbGxlZF9kaXIpCiAKLWluc3RhbGxlci5pbnN0YWxsKHVybD0iaHR0cDovL3B5cGku
cHl0aG9uLm9yZy9wYWNrYWdlcy9zb3VyY2UvbS9tZWNoYW5pemUvbWVjaGFuaXplLTAuMS4xMS56
aXAiLAoraW5zdGFsbGVyLmluc3RhbGwodXJsPSJodHRwOi8vcHlwaS5weXRob24ub3JnL3BhY2th
Z2VzL3NvdXJjZS9tL21lY2hhbml6ZS9tZWNoYW5pemUtMC4yLjQuemlwIiwKICAgICAgICAgICAg
ICAgICAgIHVybF9zdWJwYXRoPSJtZWNoYW5pemUiKQogaW5zdGFsbGVyLmluc3RhbGwodXJsPSJo
dHRwOi8vcHlwaS5weXRob24ub3JnL3BhY2thZ2VzL3NvdXJjZS9wL3BlcDgvcGVwOC0wLjUuMC50
YXIuZ3ojbWQ1PTUxMmE4MThhZjk5NzkyOTBjZDYxOWNjZThlOWMyZTJiIiwKICAgICAgICAgICAg
ICAgICAgIHVybF9zdWJwYXRoPSJwZXA4LTAuNS4wL3BlcDgucHkiKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>