voxeu.org translator not working - changed URL pattern

dapperdan
The voxeu.org translator is not working (example), because they removed www. from the default URLs. They still accept it though, i.e. they do not redirect www.voxeu.org/* to voxeu.org/*.
In short: the target regex has to be changed to ^https?://[\w\-\.]*voxeu\.org or something similar.
  • adamsmith
    thanks - I'll get to that asap (I'll change the regexp to (www\.)? which is the customary and slightly more elegant way to deal with this.).
  • dapperdan
    Sure. [\w\-\.]* may be more robust in (the admittedly very odd) case they decide to change it to something completely different again. Plus, being a regex newbie, it took me some time to figure it out. :)
    Seriously and just for my interest, is there a drawback in using [\w\-\.]*?
  • adamsmith
    we try to keep regexps pretty narrow and restrictive to avoid false positives. If I read your code correctly, it would return TRUE for a pretty much anything before voxeu - mobile.voxeu.org for example - which I don't think exists, but if it did probably wouldn't work.
  • adamsmith
    it's now up - thanks again for the pointer.

This is an old discussion that has not been active in a long time. Instead of commenting here, you should start a new discussion. If you think the content of this discussion is still relevant, you can link to it from your new discussion.