Skip to content

[web] cache the base URL as root index.html #136594

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
auto-submit merged 2 commits into from
Oct 30, 2023
Merged

[web] cache the base URL as root index.html #136594

auto-submit merged 2 commits into from
Oct 30, 2023

Conversation

@p-mazhnik
Copy link
Contributor

@p-mazhnik p-mazhnik commented Oct 15, 2023 •
edited
Loading

Fixes #136593

Caching of the base url was introduced in #53666 but resources can contain multiple index.html files, and currently hash of the latest asset will be assigned to the base url, which is not necessarily the root index.html

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Oct 15, 2023
@andrewkolos andrewkolos added the fyi-web For the attention of Web platform team label Oct 18, 2023
@andrewkolos andrewkolos self-requested a review October 18, 2023 00:15
@andrewkolos
tiny comment tweaks
9b019fc 
Tweaks some comments for additional clarity (not that they were unclear to begin with).
andrewkolos
andrewkolos approved these changes Oct 18, 2023
View reviewed changes
Copy link
Contributor

@andrewkolos andrewkolos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@andrewkolos
Copy link
Contributor

andrewkolos commented Oct 18, 2023

I plan to add a second reviewer during the next weekly tools PR review meeting this thursday.

christopherfujino
christopherfujino approved these changes Oct 30, 2023
View reviewed changes
Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@andrewkolos andrewkolos added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 30, 2023
@auto-submit auto-submit bot merged commit 0d52630 into flutter:master Oct 30, 2023
@andrewkolos
Copy link
Contributor

andrewkolos commented Oct 30, 2023

Thank you for your contribution!

@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
6620b7c 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
3445337 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
7f76dfe 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
3cbc36d 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
5be8912 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
72ba2ec 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
2afb426 
…136594)
ksokolovskyi pushed a commit to ksokolovskyi/flutter that referenced this pull request Oct 31, 2023
@p-mazhnik @ksokolovskyi
[web] cache the base URL as root index.html (flutter#136594)
8395a5c 
Fixes flutter#136593

Caching of the base url was introduced in flutter#53666 but resources can contain multiple `index.html` files, and currently hash of the **latest** asset will be assigned to the base url, which is not necessarily the root index.html
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Oct 31, 2023
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 31, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
7cbd6bd 
…136594)
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Nov 1, 2023
Merged
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 1, 2023
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
2acc152 
…136594)
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Nov 1, 2023
@engine-flutter-autoroll
Roll Flutter from e12d1a7 to 29b2516 (34 revisions) (#5290)
6ad40b9 
Roll Flutter from e12d1a7 to 29b2516 (34 revisions)

flutter/flutter@e12d1a7...29b2516

2023-11-01 [email protected] Roll Flutter Engine from 5ef5e6da9075 to 8ffc705a55d5 (2 revisions) (flutter/flutter#137664)
2023-11-01 [email protected] Roll Flutter Engine from a0ac6b432cdf to 5ef5e6da9075 (1 revision) (flutter/flutter#137657)
2023-11-01 [email protected] Roll Flutter Engine from db06c2e10459 to a0ac6b432cdf (8 revisions) (flutter/flutter#137656)
2023-11-01 [email protected] Add dependency on leak_tracker and leak_tracker_testing to flutter_test. (flutter/flutter#137646)
2023-11-01 [email protected] Roll Flutter Engine from 406b7d71ca37 to db06c2e10459 (5 revisions) (flutter/flutter#137651)
2023-11-01 [email protected] Roll pub packages (flutter/flutter#137650)
2023-10-31 [email protected] Manual roll Flutter Engine from 39be0fc34503 to 406b7d71ca37 (8 revisions) (flutter/flutter#137649)
2023-10-31 [email protected] PerformanceModeRequestHandle should dispatch creation and disposal events. (flutter/flutter#137642)
2023-10-31 [email protected] Fix minor typo in doc of window.dart (flutter/flutter#137483)
2023-10-31 [email protected] fix. typos (flutter/flutter#137178)
2023-10-31 [email protected] Document the scale argument to NetworkImage (flutter/flutter#137456)
2023-10-31 [email protected] Revert "Roll Flutter Engine from 39be0fc34503 to ec3e988099be (2 revisions)" (flutter/flutter#137647)
2023-10-31 [email protected] Move dependency on leak_tracker from dependencies to dev_dependencies in flutter_test. (flutter/flutter#137633)
2023-10-31 [email protected] Roll Flutter Engine from 39be0fc34503 to ec3e988099be (2 revisions) (flutter/flutter#137637)
2023-10-31 [email protected] Roll Flutter Engine from 4da3e5b12540 to 39be0fc34503 (4 revisions) (flutter/flutter#137635)
2023-10-31 [email protected] Update `ColorScheme.light`, `ColorScheme.dark`, `ColorScheme.highContrastLight`, & `ColorScheme.highContrastDark`  constructors docs for Material 3 (flutter/flutter#137149)
2023-10-31 [email protected] Roll Flutter Engine from 47465eb4fbe6 to 4da3e5b12540 (2 revisions) (flutter/flutter#137632)
2023-10-31 [email protected] Fix. typos (flutter/flutter#137465)
2023-10-31 [email protected] Roll Flutter Engine from 9de8146d21bf to 47465eb4fbe6 (1 revision) (flutter/flutter#137626)
2023-10-31 [email protected] Roll Flutter Engine from 6b9b806151ed to 9de8146d21bf (1 revision) (flutter/flutter#137624)
2023-10-31 [email protected] Roll Packages from 2af6954 to c9fec61 (12 revisions) (flutter/flutter#137623)
2023-10-31 [email protected] Fix divide-by-zero crash in animation_bench benchmark (flutter/flutter#137539)
2023-10-31 [email protected] Roll Flutter Engine from 11a569d97a26 to 6b9b806151ed (1 revision) (flutter/flutter#137617)
2023-10-31 [email protected] Fix formatting (flutter/flutter#137613)
2023-10-31 [email protected] Roll Flutter Engine from 5ca5cbb6efdb to 11a569d97a26 (1 revision) (flutter/flutter#137612)
2023-10-31 [email protected] Roll Flutter Engine from b4de734eb78e to 5ca5cbb6efdb (1 revision) (flutter/flutter#137610)
2023-10-31 [email protected] Roll Flutter Engine from 17eff7ab6299 to b4de734eb78e (1 revision) (flutter/flutter#137594)
2023-10-31 [email protected] Roll Flutter Engine from 44309c9f385c to 17eff7ab6299 (2 revisions) (flutter/flutter#137587)
2023-10-31 [email protected] Roll Flutter Engine from 64b0e07e2259 to 44309c9f385c (1 revision) (flutter/flutter#137584)
2023-10-30 [email protected] Roll Flutter Engine from 4f87c20b3c34 to 64b0e07e2259 (4 revisions) (flutter/flutter#137582)
2023-10-30 [email protected] Manual roll Flutter Engine from 84dcb4fb9301 to 4f87c20b3c34 (24 revisions) (flutter/flutter#137570)
2023-10-30 [email protected] [web] cache the base URL as root index.html (flutter/flutter#136594)
2023-10-30 [email protected] [flutter_tools] ensure processUtils reports exit code in ProcessExceptions (flutter/flutter#136672)
2023-10-30 [email protected] [flutter_tool] change print when selecting developer identity for code signing (flutter/flutter#136978)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

...
@p-mazhnik p-mazhnik deleted the web-service-worker-fix branch November 30, 2023 20:21
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 16, 2024
@engine-flutter-autoroll
0d52630 [web] cache the base URL as root index.html (flutter/flutter#…
11fb147 
…136594)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrewkolos andrewkolos andrewkolos approved these changes

+1 more reviewer

@christopherfujino christopherfujino christopherfujino approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

autosubmit Merge PR when tree becomes green via auto submit App fyi-web For the attention of Web platform team tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[web] base url has incorrect hash if assets contain multiple index.html

3 participants

@p-mazhnik @andrewkolos @christopherfujino