search

N4S4 / synology-api

A Python wrapper around Synology API
MIT License
368 stars 138 forks source link

Connect tot DSM7 #65

Closed Alindil closed 3 years ago

Alindil commented 3 years ago

Hi, I just upgraded my synology nas to DSM7. It seems that I can't connect anymore. image

image

Alindil commented 3 years ago

I found the error. In the connection request the version must be 7 and not 2 as it is now. But I don't know if it will work with DSM6 image

N4S4 commented 3 years ago

Hello thank you and nice catch, I am quite sure that DSM 7 had many changes, including API call, I tried to contact synology support with no success, however I did not have the chance to test it with DSM7 ad I did not upgrade yet, I will wait that things gets more stable yo run some tests. Are you able to interact with all functions once login?

Alindil commented 3 years ago

Hello, I did not test all the features but the only one I had a problem with was the connection. The other functions that I use work correctly

ajarzyn commented 3 years ago

Three ideas:

  1. It would be possibleto configure version of DSM. (+) less overhead (-) user would have to decide which API to use (which could be also good)
  2. I don't know if it's possible but it would be a good idea to discover which API version is called (+) user doesn't need to know which dsm version is he using (+) possible to upgrade dsm without upgrading scripts based on this api (-) more overhed by double call if incorrect version (if there is api to print dsm version API could just make that call first)
  3. Hybrid solution New initialization parameter would be introduced (swversion) which coudl get numerical value or "auto". Based on that API proceed with point 1 od 2.

BR, Andrew

N4S4 commented 3 years ago

@ajarzyn yes I was thinking the same, for now I will let the user choose the version, than I will work to make it automatic according to the version, will see what I can do

N4S4 commented 3 years ago

Fixed with v0.2.3