Caching the result worked previously since _finalUri was not passed between isolates with the from-to-map based structure, but causes issues when a page has redirects now since everything is passed. finalUri is used in only a few specific locations, so not caching the result won't be a big deal.
As a better solution and long term improvement, Destination should likely be immutable and not be updated with the results. I'll hopefully be able to work on that soon.
filiph
commented
1 year ago
Caching the result worked previously since
_finalUriwas not passed between isolates with the from-to-map based structure, but causes issues when a page has redirects now since everything is passed.finalUriis used in only a few specific locations, so not caching the result won't be a big deal.As a better solution and long term improvement,
Destinationshould likely be immutable and not be updated with the results. I'll hopefully be able to work on that soon.