Skip to content

Commit 247b856

Browse files
Merge pull request #24 from Tecnativa/fix-readme
[FIX] Readme.md since 443 needs to be specified when used
2 parents 7081b68 + 0eada72 commit 247b856

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

.github/workflows/pre-commit.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,6 @@ jobs:
1010
pre-commit:
1111
runs-on: ubuntu-latest
1212
steps:
13-
- uses: actions/checkout@v2
14-
- uses: actions/setup-python@v2
15-
- uses: pre-commit/action@v2.0.0
13+
- uses: actions/checkout@v4
14+
- uses: actions/setup-python@v4
15+
- uses: pre-commit/action@v3.0.1

README.md

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -110,10 +110,16 @@ Only used when [pre-resolving](#pre-resolve) is enabled.
110110

111111
### `PORT`
112112

113-
Default: `80 443`. If you're proxying HTTP/S services, no need to specify!
114-
115-
The port where this service will listen, and where the [target](#target) service is
116-
expected to be listening on also.
113+
**Default:** `80 443` Ports on which the proxy will listen and forward requests.
114+
115+
- For standard HTTP/HTTPS services, you **do not** need to change anything (the
116+
default covers both port 80 and 443).
117+
- If you only need to proxy HTTPS (or your service listens on a different port, or you
118+
want to restrict the proxy to TLS only), specify:
119+
```yaml
120+
environment:
121+
PORT: "443"
122+
```
117123
118124
### `PRE_RESOLVE`
119125

0 commit comments

Comments
 (0)