-
-
Notifications
You must be signed in to change notification settings - Fork 941
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
[Tumblr] Blogs who only accept a "https://www.tumblr.com/<user name>/<ID>/" fail to download #3628
Comments
I'll look into it, but you can also manually rewrite those URLs to put them into the format expected by gallery-dl: |
So I took a look, but I'm not really sure what the problem is supposed to be. Both
|
I have no idea what kinda sorcery is happening on my computers, but it doesn't work with either of these URLs, I'm gonna post the verbose output and the Tumblr section of the .conf file:
Maybe I should create a new API key for gallery-dl? Is it possible, that another .conf setting is somehow affecting the retrieval? Is it possible, that the python environment is somehow borked? |
https://www.tumblr.com/hismajestythebeggar That one also doesn't work (warning, NSFW) ;( |
Found the problem: Those blogs are "dashboard-only". You need all four OAuth tokens to access them:
You should see
|
I tried that, got the following URL back:
The "oauth_token" from the URL is the "access-token", and the "oauth_verifier" is the "access-token-secret", right? I tried that, but it still doesn't work ;-(
What am I doing wrong? |
- bug from 70ce45d - broke oauth:tumblr, oauth:flickr, and oauth:smugmug
You didn't do anything wrong. I made a mistake in commit 70ce45d which stopped any oauth procedure before it could finish and display the two
|
@mikf Thanks a lot, now all works perfectly! |
There are some Tumblr blogs, that won't accept a "https://<user name>.tumblr.com/post/" url, but only a "https://www.tumblr.com/<user name>/<ID>/" url. An attempt to download from these blogs fails. One of thee blogs would be https://www.tumblr.com/suf-fering. Could you take a look at it, please?
The text was updated successfully, but these errors were encountered: