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

Absolute path in Export Dir leads to issues #397

Closed
3 of 5 tasks
alexbelgium opened this issue Jan 17, 2025 · 0 comments · Fixed by #403
Closed
3 of 5 tasks

Absolute path in Export Dir leads to issues #397

alexbelgium opened this issue Jan 17, 2025 · 0 comments · Fixed by #403
Assignees
Labels
bug Something isn't working

Comments

@alexbelgium
Copy link

alexbelgium commented Jan 17, 2025

Issue Type

  • Bug report

Description

  • Setting an absolute path for the "Export Path" leads to unavailable files in the webui

Steps to Reproduce (for bugs)

  1. Set Export path to an absolute path (for example /mnt/NAS/Birdnet-Go/ to use an external drive mounted in /mnt)
  2. Go in the webui
  3. Even though the spectrogram are displayed, the files cannot play and responds with a 404 (Not Found). Example of url : http://192.168.178.23:8087/%2Fmnt%2FNAS%2FBirdnet-Go%2F2025%2F01%2Fdendrocopos_major_98p_20250117T144009Z.wav

Expected behavior

Accept usage of absolute paths, starting with /

Actual behavior

Absolute paths are accepted in the UI, but lead to impossibility to show data in the UI

Screenshots

Image

Environment

Birdnet-Go homeassistant addon accessed through ip:port (so bypassing HA's specific elements)

$ ./birdnet-go
🐦 BirdNET-Go v (built: 2025-01-15T20:45:29Z), using config file: /root/.config/birdnet-go/config.yaml

System Information

  • Device:
    • Raspberry Pi (specify model, e.g., Raspberry Pi 4 Model B)
    • Other SBC (please specify)
    • PC/Server N100 amd64 system
  • OS: HA os
  • OS Version: based on
  • Hardware specs:
    • CPU: N100intel
    • RAM: 16gb
    • Storage type and size: 256GB SSD

For Non-Raspberry Pi Linux Users

Have you tested with the latest dev branch container image?

  • Yes

Additional context

Possible Solution

not sure - add a modifier to avoid "//" in the url ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants