-
Notifications
You must be signed in to change notification settings - Fork 78
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
keep2share: fix save with correct filename with paid account #66
Conversation
Your fix doesn't work with such links:
Can you give an example of such filenames? What filenames you expect, and what filenames you get now? BTW. May I have a favor? Could you run this script with your login and password for keep2share:
and give the output of it. I just want to know if in this way I can check if this is a premium account. |
Ooops, yes I confirm so... dbg: plowdown: --no-overwrite selected On Wed, Mar 16, 2016 at 4:08 PM, Raziel-23 notifications@github.com wrote:
|
before logging on my profile from a browser
after logging in from my browser
On Wed, Mar 16, 2016 at 4:37 PM, Eddi De Pieri eddi@depieri.net wrote:
|
Thank you for testing this script. Just as I thought, for free accounts they aren't set account_expires field, but for premium accounts they do. Ok back to your problem, can you give me an example of filenames, what filenames you expect? |
on a browser I get:
but on plowdown it seems that the content type isn't text-html, so the On Wed, Mar 16, 2016 at 4:53 PM, Raziel-23 notifications@github.com wrote:
|
I'm trying to figure out how to get filename from the official keep2share api: |
Finally this api worked for me. I will prepare fix based on api. |
Please test this version: |
It works correctly! Thank you! On Wed, Mar 16, 2016 at 7:30 PM, Raziel-23 notifications@github.com wrote:
|
It's now in the master branch. |
No description provided.