docs: add credential verification guidance for Docker installs#14422
docs: add credential verification guidance for Docker installs#14422brynsofz wants to merge 1 commit into
Conversation
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @brynsofz on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added. |
There was a problem hiding this comment.
Code Review
This pull request updates the GeoNode documentation to guide users on verifying and securing credentials, handling randomly generated passwords, and fixing broken links. The review feedback correctly identifies two critical issues in the hardening instructions: first, changing the GeoServer admin password via the UI without updating the .env file will break integration; second, using docker compose restart fails to reload updated .env variables, so containers should be recreated using docker compose up -d instead.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| 2. **GeoServer admin password**: Confirm that the GeoServer admin password is not the default value and matches the value you expect from `.env`. If you need to change it: | ||
|
|
||
| - Log into GeoServer at `https://my_geonode.geonode.org/geoserver` | ||
| - Go to `Security` > `Users, Groups, and Roles` > `Users/Groups` | ||
| - Change the admin user password |
There was a problem hiding this comment.
If you change the GeoServer admin password via the GeoServer Web UI, you must also update the GEOSERVER_ADMIN_PASSWORD variable in your .env file and recreate the Django container. Otherwise, GeoNode will be unable to authenticate with GeoServer using the old password, which will break core integration features such as layer uploads, map creation, and catalog synchronization. Please update the instructions to include updating the .env file and recreating the containers.
| 4. **Restart the containers**: | ||
|
|
||
| ```bash | ||
| docker compose restart django | ||
| docker compose restart geoserver | ||
| ``` |
There was a problem hiding this comment.
Using docker compose restart does not reload the .env file or recreate the containers with the updated environment variables. As a result, the containers will continue to run with the old OAUTH2_CLIENT_ID and OAUTH2_CLIENT_SECRET values, while the database has the new ones, causing OAuth2 authentication to fail. To apply changes made to the .env file, you should recreate the containers using docker compose up -d instead.
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.