Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Allow for https - see Geonode PR 4403 #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jondoig
Copy link

@jondoig jondoig commented May 17, 2019

No description provided.

Copy link
Member

@francbartoli francbartoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jondoig, I think the control for the protocol variable has to be done within this statement

if [ ${GEONODE_LB_HOST_IP} ]
and then the variable GEONODE_LB_HOST_IP updated accordingly. Does this make sense?

@jondoig
Copy link
Author

jondoig commented May 17, 2019

Thanks Francesco - actually I don't follow, but I'm sure you're right. I appreciate your effort to lead me gently into enlightenment 😄, but I'm really just copypasting at this point. First time to do a PR, and just learning docker. So feel free to make the changes needed, or tell me exactly what I need to do.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants