-
Notifications
You must be signed in to change notification settings - Fork 348
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Checking mergeability…
Don’t worry, you can still create the pull request.
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: nginx/unit
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: nginx/unit
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1.32
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 7 commits
- 22 files changed
- 3 contributors
Commits on Mar 15, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 97ff099 - Browse repository at this point
Copy the full SHA 97ff099View commit details -
Var: Fix cacheable issue for njs variable access
The variables accessed with JS template literal should not be cacheable. Since it is parsed by njs engine, Unit can't create indexes on these variables for caching purpose. For example: { "format": "`{bodyLength:\"${vars.body_bytes_sent}\",status:\"${vars.status}\"}\n`" } The variables like the above are not cacheable. Closes: #1169
Configuration menu - View commit details
-
Copy full SHA for 6359c74 - Browse repository at this point
Copy the full SHA 6359c74View commit details -
Tests: NJS cacheable variables with access log
Reproduces issue #1169.
Configuration menu - View commit details
-
Copy full SHA for d7ff6bb - Browse repository at this point
Copy the full SHA d7ff6bbView commit details -
Wasm-wc: Fix application restarts
Liam reported a problem when trying to restart wasm-wasi-component based applications using the /control/applications/APPLICATION_NAME/restart endpoint. The application would become unresponsive. What was happening was the old application process(es) weren't exit(3)ing and so while we were starting new application processes, the old ones were still hanging around in a non-functioning state. When we are terminating an application it must call exit(3). So that's what we do. We use the return value of nxt_unit_run() as the exit status. Due to exit(3)ing we also need to now explicitly handle the return on error case. Reported-by: Liam Crilly <[email protected]> Fixes: 20ada4b ("Wasm-wc: Core of initial Wasm component model language module support") Closes: #1179 Tested-by: Liam Crilly <[email protected]> Tested-by: Danielle De Leo <[email protected]> Co-developed-by: Dan Callahan <[email protected]> Signed-off-by: Dan Callahan <[email protected]> Signed-off-by: Andrew Clayton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 777b7c8 - Browse repository at this point
Copy the full SHA 777b7c8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ca4962 - Browse repository at this point
Copy the full SHA 8ca4962View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3f228d6 - Browse repository at this point
Copy the full SHA 3f228d6View commit details
Commits on Mar 21, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 48d7917 - Browse repository at this point
Copy the full SHA 48d7917View commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...1.32