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

Minor error in README.MD? #60

Closed
pete-os opened this issue Dec 9, 2023 · 3 comments
Closed

Minor error in README.MD? #60

pete-os opened this issue Dec 9, 2023 · 3 comments

Comments

@pete-os
Copy link

pete-os commented Dec 9, 2023

Hi

Thanks for this - just installed it on my UDMP running UniFi OS 3.2.7. Not fully tested but looking go so far.

I have spotted one small issue - README.MD has an error in 'FIRST RUN' section:

cp /data/udapi-server/raddb/certs/server.pem ./raddb-server.pem
cp /data/udapi-server/raddb/certs/server-key.pem ./raddb-server-key.pem

I believe /data/udapi-server should be /data/udapi-config ?

The code (ubios-cert.env) does refer to /data/udapi-config though.

# Path to RADIUS server certificate
UBIOS_RADIUS_CERT_PATH='/data/udapi-config/raddb/certs'

Thanks

Pete

@alxwolf
Copy link
Owner

alxwolf commented Dec 11, 2023

Thanks Pete, yes, you're absolutely right! will change that.

Regards Alex

@alxwolf alxwolf closed this as completed Dec 11, 2023
@therealpaulgg
Copy link

therealpaulgg commented Dec 13, 2023

@alxwolf sorry to necro this issue..but I am not so sure that 3.2.7 is so happy with custom certificates.

While I don't use this script specifically (I have a internal CA that I use to generate certificates for my intranet), I do at the very least replace /data/unifi-core/config/unifi-core.crt along with /data/unifi-core/config/unifi-core.key. Upon restarting unifi-core with systemctl restart unifi-core, the OS overwrites my custom certificates, effectively undoing my work.

My setup has been working for months and I never had to mess with keystore or the RADIUS config (raddb). I am trying to reach out to anyone who might know more. Only reason I noticed this was because my monitoring software detected it couldn't reach my router anymore...then did some digging and found this stuff out. Router must have updated at 3am causing this to break.

@alxwolf
Copy link
Owner

alxwolf commented Dec 13, 2023

Oh, thanks for bringing this up. Then this will require some more investigation than anticipated...

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

No branches or pull requests

3 participants