NEED TESTERS (even more for DSM < 6) + feature request aggregator #26
Assignees
Labels
Comments
Required informations
Additional informationsMaintainer of the library. Feature request
|
20 tasks
|
Sure, would be happy to help !
|
|
@aaska Get this branch and open Instructions are there with comments. |
|
For the auto detect DSM version, may you paste : on your browser, while changing $host and $port. Past the result here. Thanks. |
|
Here is the result of the URL request : {"data":{"SYNO.API.Auth":{"maxVersion":4,"minVersion":1,"path":"auth.cgi"}},"success":true}Regarding populating the INFORMATION section : #28 -------------------- INFORMATION --------------------
DEBUG: Requesting URL: 'https://10.0.1.150:5001/webapi/entry.cgi?api=SYNO.DSM.Info&version=1&method=getinfo'
DEBUG: Appending access_token (SID: SID) to url
DEBUG: Request executed: 200
DEBUG: Failed: {"error":{"code":102},"success":false}
DEBUG: Error occured, retrying...
DEBUG: Requesting URL: 'https://10.0.1.150:5001/webapi/entry.cgi?api=SYNO.DSM.Info&version=1&method=getinfo'
DEBUG: Appending access_token (SID: SID) to url
DEBUG: Request executed: 200
DEBUG: Failed: {"error":{"code":102},"success":false}
-------------------- INFORMATION_END --------------------From digging a little bit, the information would need to be done through the following call : Request URL: https://xxx.yyy.fr:zzzzz/webapi/_______________________________________________________entry.cgi
Request Method: POST
[{"api":"SYNO.Core.System","method":"info","version":2,"type":"firmware"}]The output would be : {
"data": {
"has_fail": false,
"result": [
{
"api": "SYNO.Core.System",
"data": {
"firmware_date": "2018/12/26",
"firmware_ver": "DSM 5.2-5967 Update 9",
"model": "DS410j"
},
"method": "info",
"success": true,
"version": 2
}
]
},
"success": true
} |
|
@aaska Removed your public info from previous post. You never know what some people do with that info |
|
@ProtoThis Thank you so much !! Indeed, I've missed this one.. |
No problem, did check the main website (dog walking service) very nice pictures 🙂. |
|
Ahah 😃 Yeah, this is the wife's activity ! For due diligence, I've removed access from the outside world to the DSM... |
I tried the call with the following command:
but it doesn't work. Am I doing something wrong ? Simone |
|
@aaska If I can ask you one more thing, do this request on your browser : It will list all DSM API available + their path and min and max versions. Thanks. |
This was referenced Apr 12, 2020
Closed
|
I've refactor requests in a new PR #33, if anyone want to review or test, go for it ! |
|
|
|
@aaska May you test HA 0.109.0b2 (or +) ? @stast1 has a login issue home-assistant/core#34633 |
|
@chemelli74 2SA login available on HA 0.109.0 beta |
Working fine. Amazing job. Thank you !!! I only have a sensor not working: CPU. Value are completly different from those of the system monitor. Simone |
Was it the same before ? Haven’t check actually. |
Never used the lib before, all my account have 2SA enabled ;-) Simone |
|
DSM 5.2 |
|
Thanks @stast1, nice to have 👍 Actually we already do in I think your issue is caused by a missing condition here: python-synology/synology_dsm/synology_dsm.py Lines 179 to 181 in 4d9897a That is well handled while building the URL (but python-synology/synology_dsm/synology_dsm.py Lines 77 to 88 in 4d9897a |
|
First draft here : #36 |
|
Home Assistant 0.109.0b5 Logger: aiohttp.server Error handling request |
|
Thanks @stast1 May you create an issue on this repo with those comments ? Okey then, I see where it’s come from. I actually thought this was an error of not having data and success in the DSM 5 storage data payload. Will fix at lunch. |
I have multiple Synology systems in multiple locations, with backup jobs using HyperBackup, ActiveBackup for business, and Snapshot Replication for offsite. Synology offers no consolidated view of their various data protection tools. I use InfluxDB and Grafana to monitor a lot of things in my environment and my goal is to pull the various task and repository info from the syno API and feed it into Influx so I can create a unified view in Grafana. Every available API call can be found on a syno at /usr/syno/synoman/webapi and I have been able to grab exactly what I want through curl. I have a decent amount of experience working with the syno APIs through integrating Surveillance Station into Samsung SmartThings through API calls thanks to the webCore add-on to ST. I apologize in advance for my lack of experience in both python and git, but I have written a lot of shell scripts and perl in my time. Rather than build my own standalone tool, I would rather contribute to a project so others can benefit, and so I don't have to start from scratch :). My end-state target is to run this in a docker container deployed in my k8s environment. Even if I am working alone on this area for now, I hope my contributions can be useful. And am happy to provide testing on other features when I can. TIA. As a side note, I was able to get this wrapper deployed and working in my home environment quite easily, which is part of why I wanted to extend it rather than start on my own. Great work so far! |
Required informations
Additional informationsmy contributions #72 and #76 ... hope there will be more in future :) |
Quentame
pushed a commit
to Quentame/python-synology
that referenced
this issue
Aug 12, 2021
Bumps [sphinx-autobuild](https://github.com/executablebooks/sphinx-autobuild) from 2020.9.1 to 2021.3.14. - [Release notes](https://github.com/executablebooks/sphinx-autobuild/releases) - [Changelog](https://github.com/executablebooks/sphinx-autobuild/blob/main/NEWS.rst) - [Commits](https://github.com/executablebooks/sphinx-autobuild/commits) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Goal of this issue
Collecting people owning a Synology NAS to test, improve and add feature to the library
Need testers for two things :
I would like to add a DSM version "auto detect" feature to automatically discover the
dsm_versionparam, and then propably remove it.With that done, we may also add support to other DSM versions than actual 5 and 6 ! 🎉
Also we need to consolidate the test/const.py file to test DSM < 6 services 🔧
Instructions will come soon to tell you how to do so.
Registration
But first, please comment below copy-pasting this template :
(for
nothing/low/medium/high, use ctrl + B to highlight the right option)The text was updated successfully, but these errors were encountered: