Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented May 8, 2023

When using the esbuild-based browser application builder with the development server, the incoming URL for the index HTML may contain search parameters or other URL elements that can cause the index HTML content to not be found or processed incorrected by the development server. These elements are cleaned prior to comparison and the original URL is not longer passed to Vite to avoid unneeded Vite specific processing of the content.

Closes #25080

…rocessing in esbuild builder

When using the esbuild-based browser application builder with the development server, the incoming
URL for the index HTML may contain search parameters or other URL elements that can cause the index
HTML content to not be found or processed incorrected by the development server. These elements are
cleaned prior to comparison and the original URL is not longer passed to Vite to avoid unneeded
Vite specific processing of the content.
@clydin clydin added the target: patch This PR is targeted for the next patch release label May 8, 2023
@clydin clydin requested a review from alan-agius4 May 8, 2023 21:46
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label May 8, 2023
@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 May 9, 2023
@angular-robot angular-robot bot merged commit 45e98a4 into angular:main May 9, 2023
@clydin clydin deleted the esbuild/server-index-url branch May 9, 2023 14:05
@angular-automatic-lock-bot
Copy link

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 Jun 9, 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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ng serve esbuild is misreporting missing file in cli

2 participants