Increase maxRedirects to 10
Closed, ResolvedPublic0 Estimated Story Points
Actions

Assigned To

Authored By

	Mvolz
	Oct 9 2015, 12:04 PM

Description

http://dx.doi.org/10.1016/S1360-8592(98)80044-2 can't be cited as it causes a too many redirects error. We should possible increase the redirect limit, as this is resolvable with only one additional redirect.

Details

	Subject	Repo	Branch	Lines +/-
	Citoid: increase the number of redirects to 10	operations/puppet	production	+1 -1

Customize query in gerrit

Related Objects

Mentioned In: rOPUP1b6446c304eb: Citoid: increase the number of redirects to 10
rOPUPde550671442b: Citoid: increase the number of redirects to 10

Event Timeline

Mvolz created this task.Oct 9 2015, 12:04 PM

Mvolz raised the priority of this task from to Needs Triage.

Mvolz updated the task description. (Show Details)

Mvolz added a project: Citoid.

Mvolz moved this task to Site specific issues on the Citoid board.

Mvolz subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 9 2015, 12:04 PM

@mobrovac, error is "too many redirects" - do we want to increase the number of redirects?

Interestingly, this works if you just use the doi instead of using the full doi link. Not sure why that would be.

Mvolz renamed this task from http://dx.doi.org/10.1016/S1360-8592(98)80044-2 produces we couldn't make a citation for you to http://dx.doi.org/10.1016/S1360-8592(98)80044-2 causes a "too many redirects" error.Oct 10 2015, 7:03 AM

Mvolz updated the task description. (Show Details)

Mvolz set Security to None.

Hmm, weird, I tried this again and this time I got stuck in a redirect loop afterall. Will investigate more later.

Scratch that, yeah we just need to increase max redirects to get it.

Currently maxRedirects is set to 5. We can increase that to 10 without consequences.

Setting as stalled since we need an ops/puppet change for that and Opsens are away on an offsite this week. Will tackle that after that.

Sounds good; the reason the plain doi works and the link one doesn't is
because we resolve the former in requestFromDOI; it's a freebie that
doesn't count against the limit :).

Mvolz moved this task from Site specific issues to IO Tasks on the Citoid board.Oct 12 2015, 1:54 PM

Mvolz moved this task from IO Tasks to Production on the Citoid board.

• mobrovac changed the task status from Stalled to Open.Aug 15 2016, 4:00 PM

• mobrovac added projects: Services, User-mobrovac.

Restricted Application added a project: VisualEditor. · View Herald TranscriptAug 15 2016, 4:00 PM

Change 304845 had a related patch set uploaded (by Mobrovac):
Citoid: increase the number of redirects to 10

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

gerritbot added a project: Patch-For-Review.Aug 15 2016, 4:14 PM

• mobrovac mentioned this in rOPUPde550671442b: Citoid: increase the number of redirects to 10.Aug 15 2016, 4:17 PM

Jdforrester-WMF moved this task from To Triage to External and Administrivia on the VisualEditor board.Aug 15 2016, 10:11 PM

Jdforrester-WMF set the point value for this task to 0.

akosiaris mentioned this in rOPUP1b6446c304eb: Citoid: increase the number of redirects to 10.Aug 16 2016, 8:55 AM

Change 304845 merged by Alexandros Kosiaris:
Citoid: increase the number of redirects to 10