Page MenuHomePhabricator
Create Task
Maniphest T181713

Please add https//studiezaal.nijmegen.nl to $wgCopyUploadsDomains
Closed, ResolvedPublic
Actions

Description

I work as an archivist for Regionaal Archief Nijmegen.

As part of a regional Wikipedian in Residence-project I wish to upload image files to Wikimedia Commons. User Mtmlan84 is our wikipedian in residence.

Images we wish to upload have URL's like this one: https://studiezaal.nijmegen.nl/HttpHandler/icoon.ico?file=14230183

The image gallery I wish to add to Wikimedia Commons: https://studiezaal.nijmegen.nl/ran/_resultaat.aspx?invalshoek=22849198&invalshoeknaam=generiek&aantalperpagina=20&uitgebreid_zoeken=true&bsoorten_277=277&cmveldName_VrijXXzoeken=VrijXXzoeken&cmveldValue_VrijXXzoeken=%22Fotopersbureau%20Gelderland%22

No emergency, but it would be great if we could finish this project in 2017.

Details

SubjectRepoBranchLines +/-
Add https://studiezaal.nijmegen.nl to $wgCopyUploadsDomainsoperations/mediawiki-configmaster+1 -0
Customize query in gerrit

Related Objects
Search...

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.
StatusSubtypeAssignedTask
· · ·
ResolvedNoneT60224 Add domains to $wgCopyUploadsDomains (tracking)
ResolvedMarcoAurelioT181713 Please add https//studiezaal.nijmegen.nl to $wgCopyUploadsDomains
· · ·

Event Timeline

Reniervdg created this task.Nov 30 2017, 1:19 PM
Restricted Application added subscribers: Bawolff, Aklapper. · View Herald TranscriptNov 30 2017, 1:19 PM
gerritbot subscribed.Nov 30 2017, 7:41 PM

Change 394379 had a related patch set (by MarcoAurelio) published:
[operations/mediawiki-config@master] Add https://studiezaal.nijmegen.nl to $wgCopyUploadsDomains

https://gerrit.wikimedia.org/r/394379

gerritbot added a project: Patch-For-Review.Nov 30 2017, 7:41 PM
MarcoAurelio moved this task from Backlog to To deploy on the Wikimedia-Site-requests board.Nov 30 2017, 7:42 PM
MarcoAurelio moved this task from Incoming to Uploading (upload_by_url and server side upload) on the Commons board.
Framawiki added a parent task: T60224: Add domains to $wgCopyUploadsDomains (tracking).Nov 30 2017, 8:35 PM
Framawiki subscribed.
MarcoAurelio subscribed.Dec 4 2017, 12:10 PM

If there are no concerns with that domain being added I'll try to find a suitable deployment window to merge that patch.

MarcoAurelio claimed this task.Dec 6 2017, 10:05 AM

Nihil obstat so scheduling this for the next avalaible SWAT window.

Restricted Application added a project: User-MarcoAurelio. · View Herald TranscriptDec 6 2017, 10:05 AM
MarcoAurelio moved this task from unsorted/backlog to ready/patched/SWAT on the User-MarcoAurelio board.Dec 6 2017, 10:06 AM
MarcoAurelio triaged this task as Low priority.Dec 6 2017, 10:49 AM
gerritbot added a comment.Dec 6 2017, 2:31 PM

Change 394379 merged by jenkins-bot:
[operations/mediawiki-config@master] Add https://studiezaal.nijmegen.nl to $wgCopyUploadsDomains

https://gerrit.wikimedia.org/r/394379

Stashbot subscribed.Dec 6 2017, 2:33 PM

Mentioned in SAL (#wikimedia-operations) [2017-12-06T14:33:26Z] <zfilipin@tin> Synchronized wmf-config/InitialiseSettings.php: SWAT: [[gerrit:394379|Add https://studiezaal.nijmegen.nl to $wgCopyUploadsDomains (T181713)]] (duration: 00m 47s)

MarcoAurelio added a subscriber: tomasz.Dec 6 2017, 2:36 PM

Hello @Reniervdg and @tomasz:

The domain has been whitelisted now. Could you please if it is working as expected?

Thanks.

Stashbot added a comment.Dec 6 2017, 6:05 PM

Mentioned in SAL (#wikimedia-operations) [2017-12-06T18:05:34Z] <zfilipin@tin> Synchronized wmf-config/InitialiseSettings.php: SWAT: [[gerrit:394379|Add https://studiezaal.nijmegen.nl to $wgCopyUploadsDomains (T181713)]] (duration: 00m 49s)

tomasz closed this task as Resolved.Dec 6 2017, 10:57 PM

I'm not exactly sure how to test this, but as the URL has now been added to $wgCopyUploadsDomains, I'm assuming everything is working correctly.

Please do feel free to re-open this if that proves not to be the case.

MarcoAurelio moved this task from ready/patched/SWAT to cabinet on the User-MarcoAurelio board.Dec 6 2017, 11:07 PM
Reniervdg added a comment.Dec 8 2017, 10:26 AM

error upload.png (637×824 px, 61 KB)

Thank you for processing my request. I tried to test-upload an image file via URL with Pattypan, but this attempt failed. See the above attachment for the error.

Has this got anything to do with the form of URL? The URL's I am attempting to use are constructed like this: https://studiezaal.nijmegen.nl/HttpHandler/icoon.ico?file=14230183 They're not direct links that end with the image file extension (tif/jpg/etc).

MarcoAurelio added a subscriber: Reedy.Dec 8 2017, 10:30 AM

@Reniervdg I think that might be caused because of the .ico file extension which the system does not allow to upload. @Reedy any ideas on what could be done here? Thanks.

Reniervdg added a comment.Dec 8 2017, 10:39 AM

@MarcoAurelio 'ico' is part of the URL, but if you copy/paste the URL in your browser, the image file you download is a .tif

Abbe98 subscribed.Dec 11 2017, 11:34 AM

@Reniervdg are you using a custom build of Pattypan? Uploading by URL has not landed in a release yet...

Abbe98 added a comment.Jan 12 2018, 12:07 PM

@Reniervdg you can solve this by making sure the name column in your spreadsheet ends with the expected file extension, in this case .tif. This is required by Pattypan when uploading from URL.

Yarl mentioned this in T184853: Add https://audiovis.nac.gov.pl to $wgCopyUploadsDomains.Jan 13 2018, 5:27 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL