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

can't log in with DSM 4.2 #51

Closed
hoyeung3407 opened this issue Nov 18, 2017 · 7 comments
Closed

can't log in with DSM 4.2 #51

hoyeung3407 opened this issue Nov 18, 2017 · 7 comments

Comments

@hoyeung3407
Copy link

Versions

Extension version: 0.30 or 0.31, last updated 2017-11-15
Browser and version: Firefox 56.0 (64 bit)
DSM version: 4.2-3256
Download Station version: 3.3-2386

Steps to Reproduce the Issue

It just stop working some days ago,
today i open the settings page and click the "Test Connection" button,
then it shows "The requested version does not support this functionality."

Please bring back support for old DSM in future versions, thank you.

@seansfkelley
Copy link
Owner

Oh boy. Given that this happens with "test connection", this must have been caused by the fix to #32, which increased the API version used for logging in to prevent it from overriding the browser's session cookies.

It seems reasonable to reintroduce this to support old versions, with a caveat. I'm thinking:

  • synology-typescript-api should accept a version parameter for the API calls
  • The extension should check if the NAS supports a sufficient version of the auth API. If it does, change nothing. If it doesn't, drop down to using the old API version but warn the user (probably with a link to Log in with different cookie than normal browser user? #32).

Auditing synology-typescript-api for usages of version !== 1 will likely help prevent these issues in the future.

@hoyeung3407
Copy link
Author

It seems to be the solution, hope it is acceptable for other users too.
Thank you very much !

@seansfkelley seansfkelley changed the title Not work with DSM 4.2 after update can't log in with DSM 4.2 Nov 19, 2017
@seansfkelley
Copy link
Owner

@hoyeung3407 in the mean time, you can use version 0.2.1 which shouldn't have this issue (though also with fewer features). I'm hoping to get a fix out in the next couple days!

@hoyeung3407
Copy link
Author

hoyeung3407 commented Nov 20, 2017

Oh, you're very helpful. Thank you.

@seansfkelley
Copy link
Owner

Should be fixed by 6df62e8.

@hoyeung3407
Copy link
Author

You're a lifesaver !
Updated to version 0.4.1 and it works again.
Very quick response and update, thank you !

@seansfkelley
Copy link
Owner

Yay! Thanks for verifying for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants