Skip to content

fix(@angular-devkit/build-angular): correctly load dev server assets with vite 4.4.0+#25577

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:esbuild/dev-server-assets-fix
Jul 25, 2023
Merged

fix(@angular-devkit/build-angular): correctly load dev server assets with vite 4.4.0+#25577
clydin merged 1 commit intoangular:mainfrom
clydin:esbuild/dev-server-assets-fix

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Jul 24, 2023

The underlying development server (Vite) for the application build system updated
its static file serving dependencies which resulted in the @fs special file URLs
supported by Vite to have a different format. Previously, the paths were encoded
using encodeURIComponent but are now use encodeURI. The development server
integration with Vite will now use the matching encoding to allow build defined
assets to be correctly served.

@clydin clydin added the target: minor This PR is targeted for the next minor release label Jul 24, 2023
@clydin clydin force-pushed the esbuild/dev-server-assets-fix branch 3 times, most recently from e71ad65 to cc3a8fe Compare July 24, 2023 20:55
@clydin clydin marked this pull request as ready for review July 24, 2023 21:07
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jul 24, 2023
@clydin clydin requested a review from dgp1130 July 24, 2023 21:07
Comment thread tests/legacy-cli/e2e/tests/basic/serve.ts Outdated
…with vite 4.4.0+

The underlying development server (Vite) for the application build system updated
its static file serving dependencies which resulted in the `@fs` special file URLs
supported by Vite to have a different format.  Previously, the paths were encoded
using `encodeURIComponent` but are now use `encodeURI`. The development server
integration with Vite will now use the matching encoding to allow build defined
assets to be correctly served.
@clydin clydin force-pushed the esbuild/dev-server-assets-fix branch from cc3a8fe to 3e3b778 Compare July 24, 2023 23:52
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jul 25, 2023
@clydin clydin merged commit 449e21b into angular:main Jul 25, 2023
@clydin clydin deleted the esbuild/dev-server-assets-fix branch July 25, 2023 00:41
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Aug 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants