Thanks! I did some tests and LibRedirect performs much better, so I'll switch to that.
Thank you for the report. Do you have a sample redirect I could test with? I'm considering abandoning Redirector and suggesting people use LibRedirect instead. It think it's possible for it to be smarter. As a bonus, it supports multiple instances. But I'd like to try it on a link that is a known problem before I suggest it.
Hi kaki87. No news. I have a bunch of other tasks taking priority (migrating websites off of Heroku, starting a new job) so this is still on the back burner for now. If anyone is reading this and would like to take a run at it, I do have time to steer you in a direction. Feel free to post here and I'll offer what guidance I can.
Hello and thanks for the suggestion. I think this is a good idea, however it's not on my list of priorities at the moment. If you or anyone else reading this wants to give it a shot, it would probably go in the
Articles::Showaction, or in the
ArticleIdParser. In either case, it probably makes sense to break it out into a separate object as those classes already are doing enough.
This is now fixed!
Ah shoot. I've had this happen before and missed it. Thanks for the heads up.
It's unlikely I'll get to this ever. If someone is interested in making that change, I'll consider the code, but I'm going to close this for now.
Great idea. Much simpler than the lines of manual instructions. I added it in 449ece843a1e92246a25d060a29f128c4b8ddb60