Last modified: 2014-11-17 10:35:29 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T22512, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 20512 - Enable $wgAllowCopyUploads (upload by URL)
Enable $wgAllowCopyUploads (upload by URL)
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Site requests (Other open bugs)
unspecified
All All
: Normal enhancement with 5 votes (vote)
: ---
Assigned To: Ryan Kaldari
http://lists.wikimedia.org/pipermail/...
: platformeng
: 1552 5283 14919 35263 37636 (view as bug list)
Depends on: 29174 32341 42312
Blocks: 14919 28731 35545 42921 45735
  Show dependency treegraph
 
Reported: 2009-09-05 07:41 UTC by Nemo
Modified: 2014-11-17 10:35 UTC (History)
27 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Nemo 2009-09-05 07:41:27 UTC
Just a reminder / placeholder for the request by Michael Dale (see link). 
See also bug 14919.
Comment 1 Michael Dale 2009-11-03 23:28:10 UTC
Would like to bump this up now that its better supported and running successfully running on testing.wikimedia.org. 
Comment 2 Nemo 2009-12-17 07:38:22 UTC
November 5, «15:13 logmsgbot: brion synchronized php-1.5/wmf-config/InitialiseSettings.php 'enabling upload by URL for sysops on all wikis'».
December 1, «15:20 apergos: temporarily turned off upload-by-url on test.wikipedia, it interferes with testing the external page retrieval extension for fundraising».
We still have upload_by_url right on all wikis, but actual upload by URL is not enabled.
Comment 3 Michael Dale 2009-12-21 07:38:40 UTC
Any update on turning this back on? 
Comment 4 Roan Kattouw 2010-01-04 20:53:42 UTC
Enabled on test and commons; tested it and it works on both.
Comment 5 Tim Starling 2010-01-05 02:21:35 UTC
Disabled again, breaks Lucene search.
Comment 6 Michael Dale 2010-01-07 22:16:15 UTC
Fix committed in r60811 ... this re-merges new HttpFunctions.php from js2 back into trunk. Its much closer to what is currently in the deployment branch, and is needed to support async copy-by-url uploads. 

Tested locally with copy-by-url uploads. Hard to test the Lucene setup outside of the wikimedia.  
Comment 7 Roan Kattouw 2010-01-07 22:30:33 UTC
Reopening, the request itself (enable upload by URL) hasn't been fulfilled.
Comment 8 Michael Dale 2010-03-31 18:45:58 UTC
Could we at least enable $wgAllowCopyUploads without background uploading?
Comment 9 Michael Dale 2010-10-27 19:48:40 UTC
Bryon has been working on a jobQueue implementation. 

Bryon do you have any updates on how well its supported in trunk presently and if there are any outstanding issues of this feature that your aware of?
Comment 10 Bryan Tong Minh 2010-10-27 19:53:24 UTC
As far as I'm concerned all outstanding issues have been fixes.


Also, there is both a Brion and a Bryan. There is no such thing as a Bryon though.
Comment 11 Michael Dale 2010-10-27 21:58:51 UTC
Sorry "Bryan". Great to hear the outstanding issues have been fixed :) Can we add this to the list of feature to deploy soon after we review and deploy trunk? Is there a bug or road map page tracking that somewhere Roan, Rob ?
Comment 12 Michael Dale 2011-03-10 19:31:34 UTC
Flagging this bug for additional comment, now that the dust has settled from 1_17 it would be great if we could review setting $wgAllowCopyUploads to true for commons.
Comment 13 Michael Dale 2011-04-12 18:53:46 UTC
Any update on enabling this?
Comment 14 Michael Dale 2011-05-14 13:40:13 UTC
Any update on enabling this?
Comment 15 Sam Reed (reedy) 2011-08-11 14:04:16 UTC
(In reply to comment #14)
> Any update on enabling this?

Seems we've got some infrastructure issues as the apaches don't have direct external internet access...

Need to discuss with Tim
Comment 16 This, that and the other (TTO) 2011-10-30 10:02:00 UTC
Please see bug 14919 comment 5 - I didn't notice this followup bug.
Comment 17 Tim Starling 2011-12-20 05:25:44 UTC
Needs review.
Comment 18 p858snake 2012-01-16 13:27:17 UTC
*** Bug 14919 has been marked as a duplicate of this bug. ***
Comment 19 Sam Reed (reedy) 2012-02-09 22:05:55 UTC
*** Bug 1552 has been marked as a duplicate of this bug. ***
Comment 20 Sam Reed (reedy) 2012-02-09 22:21:32 UTC
*** Bug 5283 has been marked as a duplicate of this bug. ***
Comment 21 Sam Reed (reedy) 2012-02-09 22:55:52 UTC
*** Bug 1552 has been marked as a duplicate of this bug. ***
Comment 22 Sam Reed (reedy) 2012-03-16 16:12:11 UTC
*** Bug 35263 has been marked as a duplicate of this bug. ***
Comment 23 Smallman 2012-03-16 20:18:07 UTC
Any progress/updates?
Comment 24 Sam Reed (reedy) 2012-06-15 23:37:36 UTC
*** Bug 37636 has been marked as a duplicate of this bug. ***
Comment 25 Yann Forget 2012-08-07 07:24:43 UTC
Any progress/updates?
Comment 26 Ryan Kaldari 2012-08-30 23:32:15 UTC
We now have a custom Flickr uploading interface that was built as a GSoC project: https://gerrit.wikimedia.org/r/#/c/12269/

But unfortunately it depends on $wgAllowCopyUploads. It would be really nice to have this available during Wiki Loves Monuments.

Any update on getting internet access to the apaches?
Comment 27 Ryan Kaldari 2012-09-03 07:39:02 UTC
According to Tim, there is already a proxy set up called url-downloader.wikimedia.org which should allow the apaches to access the outside world. He also said that UploadFromUrl.php could use a good review, especially for error reporting. Other than that, I don't think there are any blockers for turning this back on.
Comment 28 Sumana Harihareswara 2012-09-20 22:04:39 UTC
Ryan, maybe you could file an RT ticket to get this higher on ops's priority list?  Thanks.
Comment 29 Erik Moeller 2012-09-21 18:23:13 UTC
I don't think there's necessarily any ops work involved. We need to review the upload from URL code and verify whether there's still any Lucene breakage in production; the URL downloader itself is operational according to Mark.
Comment 30 Ryan Kaldari 2012-09-25 00:28:39 UTC
Unfortunately I don't know enough about Lucene to tell what would cause problems. The impression I got from Tim is that the search issue was no longer a problem, but it would be best to get confirmation from him directly or someone familiar with Lucene.
Comment 31 Nemo 2012-09-25 05:08:03 UTC
Oren?
Comment 32 Tim Starling 2012-09-26 04:20:40 UTC
(In reply to comment #30)
> Unfortunately I don't know enough about Lucene to tell what would cause
> problems. The impression I got from Tim is that the search issue was no longer
> a problem, but it would be best to get confirmation from him directly or
> someone familiar with Lucene.

The Lucene issue was fixed with a hack in the MWSearch extension, but setting $wgHTTPProxy would still break any other Http::get() caller that is attempting to fetch things from the local intranet. I think the configuration should be split: UploadFromUrl should pass a "proxy" option to MWHttpRequest::factory() with a value derived from a global variable specific to this feature.
Comment 33 Yann Forget 2012-09-26 06:15:51 UTC
Could we bump up the priority of this? I am surprised it is still not working on Wikimedia Commons.
Comment 34 Sam Reed (reedy) 2012-09-28 00:34:12 UTC
(In reply to comment #32)
> (In reply to comment #30)
> > Unfortunately I don't know enough about Lucene to tell what would cause
> > problems. The impression I got from Tim is that the search issue was no longer
> > a problem, but it would be best to get confirmation from him directly or
> > someone familiar with Lucene.
> 
> The Lucene issue was fixed with a hack in the MWSearch extension, but setting
> $wgHTTPProxy would still break any other Http::get() caller that is attempting
> to fetch things from the local intranet. I think the configuration should be
> split: UploadFromUrl should pass a "proxy" option to MWHttpRequest::factory()
> with a value derived from a global variable specific to this feature.

https://gerrit.wikimedia.org/r/#/c/25605/
Comment 35 Sam Reed (reedy) 2012-09-28 16:23:13 UTC
Enabled on testwiki

Uploads from HTTPS sites fail with an error of:
Error fetching URL: Received HTTP code 403 from proxy after CONNECT
Comment 36 This, that and the other (TTO) 2012-09-29 01:15:06 UTC
I tried it using a script (via API), but it returned HTTP 500 when trying to print the error message :(

see http://pastebin.com/raw.php?i=NXLPRteT
Comment 37 Ryan Kaldari 2012-10-18 00:03:03 UTC
Looks like uploads from regular HTTP sites also return 403 Forbidden.
Comment 38 Ryan Kaldari 2012-10-18 00:11:38 UTC
Well, actually I can upload from http://farm1.staticflickr.com, but not http://upload.wikimedia.org
Comment 39 Sam Reed (reedy) 2012-10-18 00:44:45 UTC
Http worked before, as I did the responsible thing and imported copyrighted Google logos. Was a couple of weeks ago now.

Might need an exception in the squid config to do local requests..
Comment 40 Yann Forget 2012-10-18 08:31:47 UTC
We *need* this for this migration of Wiki Voyage/Travel to Wikimedia. There are 34,124 files to upload.
Comment 41 Sam Reed (reedy) 2012-10-18 11:30:50 UTC
No we don't. We can bulk upload them using a maintenance script
Comment 42 Ryan Kaldari 2012-10-18 21:40:49 UTC
We did need it for WLM, but that was 2 weeks ago :(

Since Flickr uploading seems to work, would anyone object to me moving forward with Flickr-only uploading for admins? This would involve turning on $wgAllowCopyUploads, turning off $wgCopyUploadsFromSpecialUpload, setting $wgCopyUploadsDomains to array( '*.flickr.com' ), finishing up https://gerrit.wikimedia.org/r/#/c/12269/ for UploadWizard, and setting $wgGroupPermissions['sysop']['upload_by_url'] to true.
Comment 43 Sam Reed (reedy) 2012-10-18 22:31:13 UTC
(In reply to comment #42)
> We did need it for WLM, but that was 2 weeks ago :(
> 
> Since Flickr uploading seems to work, would anyone object to me moving forward
> with Flickr-only uploading for admins? This would involve turning on
> $wgAllowCopyUploads, turning off $wgCopyUploadsFromSpecialUpload, setting
> $wgCopyUploadsDomains to array( '*.flickr.com' ), finishing up
> https://gerrit.wikimedia.org/r/#/c/12269/ for UploadWizard, and setting
> $wgGroupPermissions['sysop']['upload_by_url'] to true.

Seems sane to me, proxy should be alright for now...
Comment 44 Faidon Liambotis 2012-10-27 10:43:35 UTC
Reading through the comments, I don't see any pending ops work for this, so I'm removing the ops tag. If I'm mistaken, please re-add and clarify what needs to be done from our side.
Comment 45 Rd232 2012-11-13 10:26:39 UTC
Ping! Commons would love to have this, as discussed again at http://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals#Upload_by_URL
Comment 46 Ryan Kaldari 2012-11-13 19:03:28 UTC
Stay tuned...
Comment 47 Alex Monk 2012-11-20 20:10:43 UTC
Looks like Gerrit change #34357 enables this with flickr domains for commons admins.
Comment 48 Sumana Harihareswara 2012-11-27 02:43:32 UTC
Gerrit change #34357 is now merged and https://gerrit.wikimedia.org/r/#/c/25605/ is merged; ok to close this ticket?
Comment 49 This, that and the other (TTO) 2012-11-27 09:51:18 UTC
This isn't working at the moment, because it is apparently "disabled until author bug is fixed". Is this "author bug" tracked?
Comment 50 Nemo 2012-11-27 10:25:09 UTC
(In reply to comment #49)
> This isn't working at the moment, because it is apparently "disabled until
> author bug is fixed". 

aka Ie59c3eb7

> Is this "author bug" tracked?

Yes, bug 42312, and it's fixed. It's useful if you test it on [[testwiki:]] and mark it VERIFIED if ok.
Comment 51 Ryan Kaldari 2012-11-27 19:23:02 UTC
As soon as https://gerrit.wikimedia.org/r/#/c/34856/ is merged and deployed, I'll turn $wgAllowCopyUploads on and we can close this bug. We still need to file a separate bug for HTTPS support for the proxy service, but that isn't a blocker for enabling $wgAllowCopyUploads.

Also note that when $wgAllowCopyUploads is turned on, it's only going to be available in a very limited sense: for admins on Commons, via UploadWizard, from Flickr. People may want to file new bugs to change any of these individual limitations.
Comment 52 Ryan Kaldari 2012-12-07 01:15:14 UTC
$wgAllowCopyUploads turned on for Commons in https://gerrit.wikimedia.org/r/#/c/37353/

This feature is currently limited to administrators, via UploadWizard, from Flickr.com.

Feel free to file new bugs if you'd like any of these restrictions changed.
Comment 53 Yann Forget 2012-12-26 08:41:05 UTC
Upload from Flickr works for Firefox on Windows XP, but not with Chrome Version 23.0.1271.10. Should I open a new bug?
Comment 54 Ryan Kaldari 2012-12-26 10:34:09 UTC
Yes, please file a bug for that. BTW, some people have had trouble using Flickr uploading if they have the HTTPS Everywhere extension turned on, so you may want to check that.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links