Skip to content

Commit

Permalink
Merge pull request #160 from ckan/no-ckan-port
Browse files Browse the repository at this point in the history
remove broken, confusing CKAN_PORT setting
  • Loading branch information
kowh-ai authored Jul 19, 2024
2 parents a800f63 + 18fbe44 commit a1ae332
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
2 changes: 0 additions & 2 deletions .env.example
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,6 @@ USE_HTTPS_FOR_DEV=false
CKAN_VERSION=2.10.0
CKAN_SITE_ID=default
CKAN_SITE_URL=https://localhost:8443
CKAN_PORT=5000
CKAN_PORT_HOST=5000
CKAN___BEAKER__SESSION__SECRET=CHANGE_ME
# See https://docs.ckan.org/en/latest/maintaining/configuration.html#api-token-settings
CKAN___API_TOKEN__JWT__ENCODE__SECRET=string:CHANGE_ME
Expand Down
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -248,9 +248,6 @@ ckan
Add these lines to the `ckan-dev` service in the docker-compose.dev.yml file

```yaml
ports:
- "0.0.0.0:${CKAN_PORT}:5000"

stdin_open: true
tty: true
```
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ services:
- solr
- redis
ports:
- "0.0.0.0:${CKAN_PORT_HOST}:${CKAN_PORT}"
- "0.0.0.0:${CKAN_PORT_HOST}:5000"
volumes:
- ckan_storage:/var/lib/ckan
- ./src:/srv/app/src_extensions
Expand Down

0 comments on commit a1ae332

Please sign in to comment.