[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment.
If the TTL isn’t too long (I saw a cap of 1 day in the puppet config, is that correct?), then normal expiry is probably enough.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDECc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Ladsgroup
Ladsgroup added a comment.
Nope :D We need this for all clients not just Commons.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: LadsgroupCc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment.
Okay, in that case we can close this issue now, right?TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDECc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread akosiaris
akosiaris added a comment.

In T163922#3639831, @Lucas_Werkmeister_WMDE wrote:
If the TTL isn’t too long (I saw a cap of 1 day in the puppet config, is that correct?), then normal expiry is probably enough.


It doesn't work like that. The time that request can be cached is determined by the headers the response sets. http://book.varnish-software.com/3.0/HTTP.html is a pretty interesting read if you 've never read it before. It's also a rabbithole (ableit not a very big one ;-). In absence of these (like in this case where only the Age header was set) it's not easy to deduce when the page is going to be removed from all existing caches (some of which we don't really control, like the browser cache)

Anyway, I 've purged the caches in order to resolve this faster instead of waiting it out. For the interested the commands were (in that sequence)

varnishadm  ban "obj.status == 301 && req.http.host ~ commons.wikimedia.org"
varnishadm -n frontend  ban "obj.status == 301 && req.http.host ~ commons.wikimedia.org"

Do remember to force refresh to test it as your browser probably has the result cached as well.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: akosiarisCc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread elukey
elukey added a comment.

In T163922#3639504, @Lucas_Werkmeister_WMDE wrote:
Is there any way to find out which URLs are garbled? Can we look for RW_PROTO in all the cached redirects, or something like that?


There is a way (https://wikitech.wikimedia.org/wiki/Varnish#One-off_purges_.28bans.29) but it is going to be risky if we don't get the purge pattern right, since we might risk to cut too many objects from the cache. I would avoid it if possible (and wait the normal caching expiry workflow), but we can discuss with the Traffic team another approach if you want.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: elukeyCc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread daniel
daniel added a comment.
@elukey, @Lucas_Werkmeister_WMDE, @Ladsgroup Thanks for the quick fix!TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: danielCc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment.
Is there any way to find out which URLs are garbled? Can we look for RW_PROTO in all the cached redirects, or something like that?TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDECc: ema, elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread elukey
elukey added a comment.
All the appservers are now returning the good version of the redirect, I think that some of them are still showing up broken due to caching.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: elukeyCc: elukey, TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Ladsgroup
Ladsgroup added a comment.
Also I think some are behind varnish, e.g. this works fine: https://commons.wikimedia.org/data/main/Data:Amsterdam_Districts.mapTASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: LadsgroupCc: TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment.
It seems to be live on some servers and not yet on others. (And also, Varnish cashes the redirects.) I’m running this command:

until ! curl -s -I https://commons.wikimedia.org/data/main/Data:Bundestagswahl2017/wahlkreis46.map?breakCache=$RANDOM | grep -qF RW_PROTO; do sleep 60s; done; notify-send 'redirect fixed (at least when not cached)'

and it occasionally sends out notifications already.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDECc: TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread Ladsgroup
Ladsgroup added a comment.
It takes around half an hour (to an hour) to nodes to pick it up and restart.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: LadsgroupCc: TerraCodes, Jay8g, Liuxinyu970226, Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-27 Thread gerritbot
gerritbot added a comment.
Change 380774 merged by Elukey:
[operations/puppet@production] Fix /data/ redirect for commons

https://gerrit.wikimedia.org/r/380774TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: gerritbotCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread gerritbot
gerritbot added a comment.
Change 380774 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[operations/puppet@production] Fix /data/ redirect for commons

https://gerrit.wikimedia.org/r/380774TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: gerritbotCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment.
The URL https://commons.wikimedia.org/data/main/Data:Bundestagswahl2017/wahlkreis46.map now redirects to https://commons.wikimedia.org/%7bENV:RW_PROTO%7d://commons.wikimedia.org/wiki/Special:PageData/main/Data:Bundestagswahl2017/wahlkreis46.map… (%7b and %7d are { and })TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDECc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread gerritbot
gerritbot added a comment.
Change 360887 merged by Alexandros Kosiaris:
[operations/puppet@production] Add /data/ Redirect for commons

https://gerrit.wikimedia.org/r/360887TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: gerritbotCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread daniel
daniel added a comment.
*sigh* 301 it is, then. I wrote some more on the patch. Let's just ignore the pesky RFC ;)TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: danielCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread Dereckson
Dereckson added a comment.
As there is no argument for 303, we only have to sort between 301 and 302, and the main question for that is the stable or not property of the target URL.

If the redirection will be stable and always point to the same resource at the same URL, we can use a 301 (permament). If not, that will be a 302.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: DerecksonCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread daniel
daniel added a comment.
@akosiaris I replied on the patch. Basically: 301, 302, 303 are all wrong. Pick one and give us a redirect :)

Using an absolute target URL for consistency seems like a good idea, even if it's not necessary.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: danielCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread akosiaris
akosiaris added a comment.
There are other comments from yours truly in the last review, namely maintaining the status quo of configuring the redirect, aside from the 303 vs 301 part, on which I can be convinced with a good enough argument, but I haven't yet seen a reply.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: akosiarisCc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T163922: Create a URL rewrite to handle the /data/ path for canonical URLs for machine readable page content

2017-09-26 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment.
Is this just blocked on the question of HTTP 301 vs. 303, which is still open on Gerrit? Or is there something else? We should really get this redirect in place, we’ve already been exposing /data/ URLs in RDF exports and the query service for a while now.TASK DETAILhttps://phabricator.wikimedia.org/T163922EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDECc: Lucas_Werkmeister_WMDE, Dereckson, akosiaris, fgiunchedi, Stashbot, gerritbot, Ladsgroup, Aklapper, daniel, Lordiis, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, Liugev6, Zoranzoki21, Lewizho99, Maathavan, DatGuy, Devwaker, Urbanecm, JEumerus, Tulsi_Bhagat, suriyaa, Luke081515, biplabanand, Wikidata-bugs, Snowolf, Dcljr, Southparkfan, Jdforrester-WMF, Matanya, Rxy, Jay8g, Glaisher, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs