marcopeocchi / yt-dlp-web-ui

A terrible web ui for yt-dlp. Designed to be self-hosted.
Mozilla Public License 2.0
657 stars 67 forks source link

Host and port set in config file are not respected #129

Open gilliginsisland opened 5 months ago

gilliginsisland commented 5 months ago

This line server.RunBlocking(host, port, frontend, localDatabasePath) in main.go is not reading the host and port from the config file, it should be changed to server.RunBlocking(c.host, c.port, frontend, localDatabasePath).

Also would be nice if localDatabasePath was added to the config file similar to how session_file_path and log_path are.

marcopeocchi commented 5 months ago

hello @gilliginsisland,

if you're using yt-dlp-webui with docker is normal behavior, just use Docker's NAT. If you're using without Docker then there's a problem that needs to be addressed.

About localDatabasePath it can be added without problems.

gilliginsisland commented 4 months ago

I'm not using docker, I am running the binary directly. When passing host and port via cli args it works fine, its just through the config file that it doesn't work.