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

Error 404 #1

Open
deese opened this issue Nov 10, 2023 · 11 comments
Open

Error 404 #1

deese opened this issue Nov 10, 2023 · 11 comments

Comments

@deese
Copy link

deese commented Nov 10, 2023

Hi! Im trying to deploy your add-on and I'm getting a 404 when I try to connect. did you experienced this issue?
Thanks! :D

@bilogic
Copy link
Owner

bilogic commented Nov 11, 2023

Yes, it won't be able to load in HA, perhaps I should remove the side bar for now.
It is due to this issue navidrome/navidrome#248

What I do is to use Caddy to expose Navidrome as HTTPS.

http://your-navidrome.com {
    redir https://{host}{uri}
}

http://your-navidrome.com {
    # Custom SSL Conf
    tls /ssl/your-navidrome.com.fullchain.crt /ssl/your-navidrome.com.key
    
    reverse_proxy * 127.0.0.1:4533 {
        header_up Host {http.reverse_proxy.upstream.hostport}
        header_up X-Forwarded-For {http.request.remote}
        header_up X-Real-IP {http.reverse_proxy.upstream.port}
    }
}

@bilogic
Copy link
Owner

bilogic commented Nov 11, 2023

@deese

I think ND_BASEURL navidrome/navidrome#103 might solve the issue, but I don't have time to look into it yet.

@deese
Copy link
Author

deese commented Nov 11, 2023

Thanks for the quick reply. I was playing a bit with the ND_BASEURL and it didn't work. I will try again to see if I'm able to do it.
Thanks!

@bilogic
Copy link
Owner

bilogic commented Nov 12, 2023

I don't think ND_BASEURL was exposed in my repo and Navidrome's redirect is kind of broken

Jellyfin

image

Navidrome, needs to be (./app)

image

@bilogic
Copy link
Owner

bilogic commented Nov 12, 2023

See navidrome/navidrome#2605

@deese
Copy link
Author

deese commented Nov 12, 2023

Don't know if it's the best way to do that but I added this to "rootfs/etc/cont-init.d/90-data_location.sh" to export the ND_BASEURL and the ND_LOGLEVEL

bashio::config.has_value 'ND_BASEURL' && export ND_BASEURL=$(bashio::config 'ND_BASEURL') && bashio::log.info "ND_BASEURL set to $ND_BASEURL"
bashio::config.has_value 'ND_LOGLEVEL' && export ND_LOGLEVEL=$(bashio::config 'ND_LOGLEVEL') && bashio::log.info "ND_LOGLEVEL set to $ND_LOGLEVEL"

@bilogic
Copy link
Owner

bilogic commented Nov 13, 2023

yes, that seems about right

@bilogic
Copy link
Owner

bilogic commented Nov 13, 2023

oh, you will have to update config.json too

@deese
Copy link
Author

deese commented Nov 13, 2023

Yep, sorry. I also did that haha.

@bilogic
Copy link
Owner

bilogic commented Apr 26, 2024

@deese did you manage to navidrome to run in HA? If yes, can PR back to me? Thanks!

@bilogic
Copy link
Owner

bilogic commented May 11, 2024

@deese the value of ND_BASEURL might have to be ., is this something you have some time to try?
I also don't think we should be exposing ND_BASEURL since setting it to any other value will make it incompatible with HA.

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

2 participants