ckan / ckan-docker

Scripts and images to run CKAN using Docker Compose
104 stars 193 forks source link

remove broken, confusing CKAN_PORT setting #160

Closed wardi closed 4 months ago

wardi commented 4 months ago

changing the value of CKAN_PORT results in a broken environment because it's not passed to ckan run

Instead of passing CKAN_PORT to ckan run this PR removes the setting because there's no reason to change the port inside the container. CKAN_PORT_HOST allows changing the mapped port outside the container which is what users will care about.