added PORT to the curl command. This makes the healthcheck send requests to the correct port. the expression returns 8080 if PORT is unspecified
curl silently fails and return no output if openwebui was down so I added -n option to jq to return non-zero exit code if the input was null/empty. stackoverflowjq github issue
Why/How I stumbled upon this problem:
I ran open-webui with PORT set to 80 (I like using tailscale to avoid having to painfully deal with reverse proxies) yet docker reported that container was healthy
Added
Changed
Deprecated
Removed
Fixed
Fixed healthcheck
Security
Breaking Changes
Additional Information
Screenshots or Videos
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/open-webui/open-webui/pull/3998
**Author:** [@ScribblerCoder](https://github.com/ScribblerCoder)
**Created:** 7/19/2024
**Status:** ✅ Merged
**Merged:** 8/2/2024
**Merged by:** [@tjbck](https://github.com/tjbck)
**Base:** `dev` ← **Head:** `patch-1`
---
### 📝 Commits (1)
- [`175d1cc`](https://github.com/open-webui/open-webui/commit/175d1cc8f27e3bbb21cc8f91fce35c6f35dd9f18) patch Healtcheck
### 📊 Changes
**1 file changed** (+1 additions, -1 deletions)
<details>
<summary>View changed files</summary>
📝 `Dockerfile` (+1 -1)
</details>
### 📄 Description
# Changelog Entry
### Description
- added `PORT` to the curl command. This makes the healthcheck send requests to the correct [port](https://docs.openwebui.com/getting-started/env-configuration#port). the expression returns 8080 if `PORT` is unspecified
- curl silently fails and return no output if openwebui was down so I added `-n` option to `jq` to return non-zero exit code if the input was null/empty. [stackoverflow](https://stackoverflow.com/questions/66468950/exit-with-non-zero-if-input-is-empty) [jq github issue](https://github.com/jqlang/jq/issues/1497)
### Why/How I stumbled upon this problem:
I ran open-webui with `PORT` set to 80 (I like using tailscale to avoid having to painfully deal with reverse proxies) yet docker reported that container was healthy
### Added
### Changed
### Deprecated
### Removed
### Fixed
- Fixed healthcheck
### Security
### Breaking Changes
---
### Additional Information
### Screenshots or Videos
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/3998
Author: @ScribblerCoder
Created: 7/19/2024
Status: ✅ Merged
Merged: 8/2/2024
Merged by: @tjbck
Base:
dev← Head:patch-1📝 Commits (1)
175d1ccpatch Healtcheck📊 Changes
1 file changed (+1 additions, -1 deletions)
View changed files
📝
Dockerfile(+1 -1)📄 Description
Changelog Entry
Description
PORTto the curl command. This makes the healthcheck send requests to the correct port. the expression returns 8080 ifPORTis unspecified-noption tojqto return non-zero exit code if the input was null/empty. stackoverflow jq github issueWhy/How I stumbled upon this problem:
I ran open-webui with
PORTset to 80 (I like using tailscale to avoid having to painfully deal with reverse proxies) yet docker reported that container was healthyAdded
Changed
Deprecated
Removed
Fixed
Security
Breaking Changes
Additional Information
Screenshots or Videos
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.