search

getmango / Mango

Mango is a self-hosted manga server and web reader
https://getmango.app
MIT License
1.69k stars 120 forks source link

[Bug Report] Download not added to queue #275

Closed NaruZosa closed 2 years ago

NaruZosa commented 2 years ago

Describe the bug Using the nHentai plugin, a specific 'manga' cannot be downloaded. Unknown if this affects others. When attempting to download, the following message is received: "0 of 1 chapters added to the download queue. You can view and manage your download queue on the download manager page."

To Reproduce Steps to reproduce the behavior:

  1. Go to the Plugins page and select nHentai
  2. Enter 'https://nhentai.net/g/244327/' as the address
  3. Select the result and attempt to download it
  4. See error

Expected behavior Download should add to the queue

Environment (please complete the following information):

Docker Default values, other than:

Additional context image image image

When this failed, I tried downloading something else from nHentai, and it was successful. I then re-attempted this and it failed again.

Debug logs show nothing of value:

[96m[DEBUG] [0m2022/01/31 13:41:23 | Setting proxy
[96m[DEBUG] [0m2022/01/31 13:41:23 | Setting read timeout
[96m[DEBUG] [0m2022/01/31 13:41:23 | Requesting https://nhentai.net/g/244327/
[96m[DEBUG] [0m2022/01/31 13:41:23 | Performing request
[96m[DEBUG] [0m2022/01/31 13:41:24 | 200 GET /api/admin/plugin/list?query=https%3A%2F%2Fnhentai.net%2Fg%2F244327%2F&plugin=nhentai 451.43ms
[96m[DEBUG] [0m2022/01/31 13:41:27 | 200 POST /api/admin/plugin/download 15.77ms
hkalexling commented 2 years ago

It's probably because you have a job with the same ID in the download queue. Can you delete the job and then try again?

NaruZosa commented 2 years ago

🤦 Sorry, not sure how that happened, closing the issue.

Would it be possible to have another line that says "X chapters already in download queue"?

On the note of deleting the job, it seems I can't delete completed downloads from the queue either. Updating the title of my other issue to reflect this.

hkalexling commented 2 years ago

Yeah I agree we can improve the message a bit. Will take a look. Thanks for the suggestion.