Skip to content

Add site server process #19

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

Merged
fluiddot merged 19 commits into from
May 8, 2024
Merged

Add site server process #19

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
f144692
Add site server process
fluiddot Apr 23, 2024
d37aecc
Update `getWpNowPath` to support site server process
fluiddot Apr 23, 2024
196c599
Update webpack configuration to generate site server process bundle
fluiddot Apr 23, 2024
e52bc0e
Use server process in the app
fluiddot Apr 23, 2024
1de5f35
Update `phpGetThemeDetails` to use server process
fluiddot Apr 23, 2024
9b54b8c
Ensure server uses port defined by the app
fluiddot Apr 23, 2024
fc7b220
Extend logging to support forked processes
fluiddot Apr 24, 2024
f69e139
Setup logging for site server process
fluiddot Apr 24, 2024
e3740cd
Improve process exit error message
fluiddot Apr 24, 2024
0f3e7db
Update webpack main configuration to support extra entries
fluiddot Apr 24, 2024
713cb65
Merge branch 'trunk' into add/site-server-process
fluiddot Apr 24, 2024
12509f3
Fix error case when waiting for a message response
fluiddot Apr 25, 2024
4a2969c
Rename wait for response function
fluiddot Apr 25, 2024
fb09b9a
Rename message payload in site server process child
fluiddot Apr 25, 2024
43846c7
Update inline comments in main webpack configuration
fluiddot Apr 25, 2024
3f7c463
Merge branch 'trunk' into add/site-server-process
fluiddot Apr 25, 2024
3566662
Use handlers approach in server child process
fluiddot May 6, 2024
8e17947
Kill process upon server stop
fluiddot May 7, 2024
9909cf5
Wait for server to stop
fluiddot May 7, 2024
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Ensure server uses port defined by the app
  • Loading branch information
@fluiddot
fluiddot committed Apr 24, 2024
commit 9b54b8cb143dbdec01f81dabf665c8f2653ec73d
2 changes: 1 addition & 1 deletion vendor/wp-now/src/start-server.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export async function startServer(
app.use(fileUpload({ defCharset: 'utf8', defParamCharset: 'utf8'}));
app.use(compression({ filter: shouldCompress }));
app.use(addTrailingSlash('/wp-admin'));
const port = await portFinder.getOpenPort();
const port = options.port ?? await portFinder.getOpenPort();
const { php, options: wpNowOptions } = await startWPNow(options);

app.use('/', async (req, res) => {
Expand Down