Skip to content

Conversation

alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Oct 29, 2024

This commit implements the capability for the App Engine to serve prerendered pages directly. Previously, we relied on frameworks like Express for this functionality, which resulted in inconsistent redirects for directories where in some cases a trailing slash was added to the route.

Note: This change applies only when using the new SSR APIs. When using the CommonEngine, a 3rd party static serve middleware is still required.

@alan-agius4 alan-agius4 force-pushed the serve-static branch 7 times, most recently from 065dcc5 to 27d92dd Compare October 29, 2024 15:58
@alan-agius4 alan-agius4 added the target: rc This PR is targeted for the next release-candidate label Oct 29, 2024
@alan-agius4 alan-agius4 force-pushed the serve-static branch 2 times, most recently from 6f56cae to f153cd1 Compare October 29, 2024 17:38
@alan-agius4 alan-agius4 requested a review from dgp1130 October 29, 2024 18:16
@alan-agius4 alan-agius4 marked this pull request as ready for review October 29, 2024 18:16
@alan-agius4 alan-agius4 removed the target: rc This PR is targeted for the next release-candidate label Oct 29, 2024
@alan-agius4 alan-agius4 added this to the v19 Candidates milestone Oct 30, 2024
Copy link
Collaborator

@dgp1130 dgp1130 left a comment

Choose a reason for hiding this comment

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

Left a few comments, the two main concerns are whether we can simplify the API surface of AngularAppEngine and how we should let devs interact with the caching policy.

@alan-agius4 alan-agius4 force-pushed the serve-static branch 2 times, most recently from 94396da to aa236be Compare October 31, 2024 08:44
Copy link
Collaborator

@dgp1130 dgp1130 left a comment

Choose a reason for hiding this comment

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

I gave some suggestions for the process, serve, render naming, but ultimately I'm happy to defer to you on what you think is best here.

@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate action: merge The PR is ready for merge by the caretaker labels Oct 31, 2024
@alan-agius4 alan-agius4 force-pushed the serve-static branch 2 times, most recently from a9ab5e7 to e640320 Compare October 31, 2024 16:24
@alan-agius4 alan-agius4 linked an issue Oct 31, 2024 that may be closed by this pull request
@alan-agius4 alan-agius4 force-pushed the serve-static branch 2 times, most recently from 1a6c78b to b85f5d9 Compare October 31, 2024 16:50
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: merge The PR is ready for merge by the caretaker labels Oct 31, 2024
This commit implements the capability for the App Engine to serve prerendered pages directly. Previously, we relied on frameworks like Express for this functionality, which resulted in inconsistent redirects for directories where in some cases a trailing slash was added to the route.

**Note:** This change applies only when using the new SSR APIs. When using the `CommonEngine`, a 3rd party static serve middleware is still required.
@alan-agius4 alan-agius4 merged commit 481ccdb into angular:main Nov 1, 2024
30 checks passed
@alan-agius4
Copy link
Collaborator Author

The changes were merged into the following branches: main, 19.0.x

@alan-agius4 alan-agius4 deleted the serve-static branch November 1, 2024 09:45
@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 Dec 2, 2024
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 area: @angular/ssr target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SSR: How to disable url normalization?
2 participants