There was a @todo in RouteNormalizerRequestSubscriber pending a fix on #2761639: PageCache should not use $request->getUri(), which has long since been committed. This removes the note and related code.

CommentFileSizeAuthor
redirect-removetodo.patch961 bytesbradjones1

Comments

bradjones1 created an issue. See original summary.

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Status: Needs review » Reviewed & tested by the community

Tested the patch and I totally agree this todo should be removed since #2761639 issue is solved.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

  • Berdir committed c0067ec on 8.x-1.x authored by bradjones1 
    Issue #2956841 by bradjones1: Remove @todo'd code in...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.