Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update unifi_ssl_import.sh #42

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

Conversation

skynett81
Copy link

did add :

Uncomment following three lines for Unifi Dream Machine CloudKey

UNIFI_DIR=/mnt/data/unifi
JAVA_DIR=/mnt/data/unifi
KEYSTORE=${JAVA_DIR}/data/keystore

did add :

# Uncomment following three lines for Unifi Dream Machine CloudKey
UNIFI_DIR=/mnt/data/unifi
JAVA_DIR=/mnt/data/unifi
KEYSTORE=${JAVA_DIR}/data/keystore
@solbu
Copy link

solbu commented Jan 14, 2020

This is wrong.
In my opinion, that change is for the user to use. It does not belong in a utility meant for a variety of distros.

@skynett81
Copy link
Author

skynett81 commented Jan 15, 2020 via email

@skynett81
Copy link
Author

but you also find it under /usr/lib/unifi too

@TheGo2SWATking
Copy link

TheGo2SWATking commented Jan 15, 2020

The point is that this script is intended to be used anywhere the unifi controller is installed. I use it on ubuntu, someone else might use it on centos, and others are using it on the cloud key and UDM. It comes with configuration lines for all of those scenarios. All you have to do is comment out the ones for centos and uncomment the cloud key lines.

Whatever your scenario is, you'll need to do some amount of configuration to get this working, it's not something you can just drop in. The hostname also needs to be changed, and if using lets encrypt you'll need to install certbot and change LE_MODE to yes. If you're using normal certificates you need to tell it where those certs are so it can import them. Determining what platform you're on and using the config lines for that platform is probably the easiest part.

Edit: I see the UDM install location is different from the cloud key. It could be added as an option, but it shouldn't be the default since there are so few of them out there.

@andybjackson
Copy link

Skynett81,

If the Unifi Dream Machine does indeed locate the files differently (I don't know as I don't have one), the your proposed changes are valid, but should be commented out (i.e. preceded by "# ") to be consistent.

Regards and thanks
Andy

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

Successfully merging this pull request may close these issues.

4 participants