• Stan Hu's avatar
    Fix 404 page when viewing TODOs that contain milestones or labels in different projects · f8a3344d
    Stan Hu authored
    A user viewing the TODOs page will see a 404 if there are mentioned labels
    in multiple different projects. This is likely a caching bug and only occurs
    when Markdown rendering occurs across multiple projects, which is why it's so
    tricky to reproduce. This is what I think is happening:
    
    1. LabelReferenceFilter#references_in encounters label ~X for ProjectA and finds the label in the DB as id = 1.
    2. LabelReferenceFilter.references_in yields [1, 'X', nil, ...]
    3. Since project_ref is nil, AbstractReferenceFilter#project_from_ref_cache caches nil => ProjectA.
    4. LabelReferenceFilter#references_in encounters label ~Y for ProjectB and finds the label in the DB as id = 2.
    5. LabelReferenceFilter.references_in yields [2, 'Y', nil, ...]
    6. AbstractReferenceFilter#project_from_ref_cache lookups nil and returns ProjectA. It was supposed to be ProjectB.
    7. A is the wrong project, so the label lookup fails.
    
    This MR caches Markdown references if the key is present.
    
    Closes #17898
    f8a3344d
todos_spec.rb 3.13 KB