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

Synology Photos not working #104837

Closed
sahara101 opened this issue Nov 30, 2023 · 3 comments · Fixed by #104848
Closed

Synology Photos not working #104837

sahara101 opened this issue Nov 30, 2023 · 3 comments · Fixed by #104848

Comments

@sahara101
Copy link

The problem

Hello, I am receiving unknown error when trying synology photos integraation. I tried both core and docker. Admin user does not have 2FA enabled.

What version of Home Assistant Core has the issue?

core-2023.11.3

What was the last working version of Home Assistant Core?

What type of installation are you running?

Home Assistant Core

Integration causing the issue

Synology DSM

Link to integration documentation on our website

No response

Diagnostics information

home-assistant_synology_dsm_2023-11-30T19-44-56.302Z.log

Example YAML snippet

No response

Anything in the logs that might be useful for us?

[140345243671104] Error handling message: Unknown error (unknown_error) admin from 192.168.1.150 (Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.1 Safari/605.1.15)

Additional information

No response

@home-assistant
Copy link

Hey there @hacf-fr, @Quentame, @mib1185, mind taking a look at this issue as it has been labeled with an integration (synology_dsm) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of synology_dsm can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign synology_dsm Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


synology_dsm documentation
synology_dsm source
(message by IssueLinks)

@mib1185
Copy link
Contributor

mib1185 commented Nov 30, 2023

final error:

  File "/usr/src/homeassistant/homeassistant/components/synology_dsm/media_source.py", line 156, in _async_build_diskstations
    assert isinstance(mime_type, str)
AssertionError

caused by DNG files, which is not an official IANA registered mime type

we may could fallback to mimetype application/octet-stream in case the original mimetype could not be guessed, but i'm not sure if this is a good approach 🤔

@sahara101
Copy link
Author

Thanks mib1185!

@github-actions github-actions bot locked and limited conversation to collaborators Dec 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants