Skip to content

Commit

Permalink
Fix missing replacement of geoserver public location (#4215)
Browse files Browse the repository at this point in the history
* Fix missing replacement of geoserver public location

* Fix diff visualization
  • Loading branch information
francbartoli committed Feb 11, 2019
1 parent 312d59f commit a5311bd
Show file tree
Hide file tree
Showing 11 changed files with 17 additions and 17 deletions.
4 changes: 2 additions & 2 deletions docker-compose.override.localhost.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ services:
- GEONODE_LB_PORT=80
- SITEURL=http://localhost/
- ALLOWED_HOSTS=['localhost', ]
- GEOSERVER_PUBLIC_LOCATION=http://localhost/geoserver/
- GEOSERVER_PUBLIC_LOCATION=http://localhost/gs/

celery:
build: .
Expand All @@ -26,7 +26,7 @@ services:
- GEONODE_LB_PORT=80
- SITEURL=http://localhost/
- ALLOWED_HOSTS=['localhost', ]
- GEOSERVER_PUBLIC_LOCATION=http://localhost/geoserver/
- GEOSERVER_PUBLIC_LOCATION=http://localhost/gs/

geoserver:
environment:
Expand Down
4 changes: 2 additions & 2 deletions docker-compose.override.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ services:
- GEONODE_LB_PORT=80
- SITEURL=http://localhost/
- ALLOWED_HOSTS=['localhost', ]
- GEOSERVER_PUBLIC_LOCATION=http://localhost/geoserver/
- GEOSERVER_PUBLIC_LOCATION=http://localhost/gs/

celery:
build: .
Expand All @@ -26,7 +26,7 @@ services:
- GEONODE_LB_PORT=80
- SITEURL=http://localhost/
- ALLOWED_HOSTS=['localhost', ]
- GEOSERVER_PUBLIC_LOCATION=http://localhost/geoserver/
- GEOSERVER_PUBLIC_LOCATION=http://localhost/gs/

geoserver:
environment:
Expand Down
2 changes: 1 addition & 1 deletion docs/reference/worldmap.txt
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ Set the following environment variables as needed (change SITE_NAME and SERVER_I
export OWNER=$PG_USERNAME
export ALLOWED_HOSTS="localhost, $SERVER_IP, "
export GEOSERVER_LOCATION=http://localhost:8080/geoserver/
export GEOSERVER_PUBLIC_LOCATION=http://$SERVER_IP/geoserver/
export GEOSERVER_PUBLIC_LOCATION=http://$SERVER_IP/gs/
export SOLR_URL =http://localhost:8983/solr/hypermap/select/
export HYPERMAP_REGISTRY_URL =http://localhost:8001
export MAPPROXY_URL=http://localhost:8001
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,7 @@ Revert to default site settings
#)

#GEOSERVER_PUBLIC_LOCATION = os.getenv(
# 'GEOSERVER_PUBLIC_LOCATION', '{}/geoserver/'.format(SITEURL)
# 'GEOSERVER_PUBLIC_LOCATION', '{}/gs/'.format(SITEURL)
#)
...

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ Preliminary Steps & Checks
)

GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', '{}/geoserver/'.format(SITEURL)
'GEOSERVER_PUBLIC_LOCATION', '{}/gs/'.format(SITEURL)
)
...

Expand Down Expand Up @@ -309,7 +309,7 @@ In order to do that, edit the ``my_geonode/local_settings.py`` file:
)

GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', '{}/geoserver/'.format(SITEURL)
'GEOSERVER_PUBLIC_LOCATION', '{}/gs/'.format(SITEURL)
)
...
CATALOGUE = {
Expand Down Expand Up @@ -475,7 +475,7 @@ In order to do that, edit the ``my_geonode/local_settings.py`` file:
)

GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', '{}/geoserver/'.format(SITEURL)
'GEOSERVER_PUBLIC_LOCATION', '{}/gs/'.format(SITEURL)
)

OGC_SERVER_DEFAULT_USER = os.getenv(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ Upgrade Development Environment
65 GEOSERVER_PUBLIC_LOCATION = os.getenv( 49 GEOSERVER_PUBLIC_LOCATION = os.getenv(
------------------------------------------------------------------------
66 # 'GEOSERVER_PUBLIC_LOCATION', '{}geoserver/'.format(SITEURL) <>
67 'GEOSERVER_LOCATION', 'http://localhost:8080/geoserver/' 50 'GEOSERVER_PUBLIC_LOCATION', 'http://localhost:8080/geoserver/'
67 'GEOSERVER_LOCATION', 'http://localhost:8080/geoserver/' 50 'GEOSERVER_PUBLIC_LOCATION', 'http://localhost:8080/gs/'
68 )
------------------------------------------------------------------------
69 =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ services:
- UWSGI_CMD=uwsgi --ini /usr/src/app/uwsgi.ini
- IS_CELERY=false
- C_FORCE_ROOT=1
- GEOSERVER_PUBLIC_LOCATION=http://geonode/geoserver/
- GEOSERVER_PUBLIC_LOCATION=http://geonode/gs/
- GEOSERVER_LOCATION=http://geonode/geoserver/
- SITEURL=http://geonode/

Expand Down Expand Up @@ -116,7 +116,7 @@ services:
- CELERY_CMD=celery worker --app=geonode.celery_app:app -B -l INFO
- IS_CELERY=true
- C_FORCE_ROOT=1
- GEOSERVER_PUBLIC_LOCATION=http://geonode/geoserver/
- GEOSERVER_PUBLIC_LOCATION=http://geonode/gs/
- GEOSERVER_LOCATION=http://geonode/geoserver/
- SITEURL=http://geonode/

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ Change the value of the `LOCATION` and `PUBLIC_LOCATION` variables as below::
'GEOSERVER_LOCATION', 'http://localhost/geoserver/'
)
GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', 'http://localhost/geoserver/'
'GEOSERVER_PUBLIC_LOCATION', 'http://localhost/gs/'
)

Finally configure GeoServer datastore:
Expand Down Expand Up @@ -167,7 +167,7 @@ The resulting configuration file should look like this:::
'GEOSERVER_LOCATION', 'http://localhost/geoserver/'
)
GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', 'http://localhost:8001/geoserver/'
'GEOSERVER_PUBLIC_LOCATION', 'http://localhost:8001/gs/'
)

# OGC (WMS/WFS/WCS) Server Settings
Expand Down
2 changes: 1 addition & 1 deletion geonode/local_settings.py.psql_test
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ GEOSERVER_LOCATION = os.getenv(
'GEOSERVER_LOCATION', 'http://localhost:8080/geoserver/'
)
GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', 'http://localhost:8080/geoserver/'
'GEOSERVER_PUBLIC_LOCATION', 'http://localhost:8080/gs/'
)

# OGC (WMS/WFS/WCS) Server Settings
Expand Down
2 changes: 1 addition & 1 deletion package/support/geonode.local_settings
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ GEOSERVER_LOCATION = os.getenv(
)

GEOSERVER_PUBLIC_LOCATION = os.getenv(
'GEOSERVER_PUBLIC_LOCATION', '{}geoserver/'.format(SITEURL)
'GEOSERVER_PUBLIC_LOCATION', '{}gs/'.format(SITEURL)
)

OGC_SERVER_DEFAULT_USER = os.getenv(
Expand Down
2 changes: 1 addition & 1 deletion tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ def update(ctx):
'GEONODE_LB_PORT'
):
ctx.run("echo export GEOSERVER_PUBLIC_LOCATION=\
http://{public_fqdn}/geoserver/ >> {override_fn}".format(**envs), pty=True)
http://{public_fqdn}/gs/ >> {override_fn}".format(**envs), pty=True)
ctx.run("echo export SITEURL=\
http://{public_fqdn}/ >> {override_fn}".format(**envs), pty=True)

Expand Down

0 comments on commit a5311bd

Please sign in to comment.