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.

Sign up for GitHub

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

Jump to bottom

fix(qbittorrent): fix two file transferring related bugs #3501

Merged
merged 2 commits into from
Feb 18, 2023
Merged

fix(qbittorrent): fix two file transferring related bugs #3501

merged 2 commits into from
Feb 18, 2023

Conversation

kdxcxs
Copy link
Contributor

@kdxcxs kdxcxs commented Feb 18, 2023

  • delete qbittorrent task but do not delete the files before transferring to avoid qbittorrent accessing downloaded files and throw cannot access the file because it is being used by another process error
  • parse the path correctly when the torrent contains folders

@kdxcxs
Copy link
Contributor Author

kdxcxs commented Feb 18, 2023

#3413 (comment)

@anwen-anyi
Copy link
Contributor

QAQ 我是不是可以用了？哈哈 我去试试看这次还行吗

@kdxcxs
Copy link
Contributor Author

kdxcxs commented Feb 18, 2023

QAQ 我是不是可以用了？哈哈 我去试试看这次还行吗

这个问题应该是种子里面有文件夹的时候没有正确解析路径，现在应该解决了。但是之前那个一直等 qb 解析直到超时的问题还没找到😭

@anwen-anyi
Copy link
Contributor

问题不大我先试试能不能下载下来~

@anwen-anyi
Copy link
Contributor

不过好像还是一样的问题QAQ
我用的这个 期待其他人测试试试看
https://github.com/alist-org/alist/actions/runs/4210480253

@kdxcxs
Copy link
Contributor Author

kdxcxs commented Feb 18, 2023

不过好像还是一样的问题QAQ 我用的这个 期待其他人测试试试看 alist-org/alist/actions/runs/4210480253

还是 The system cannot find the path specified. ？

@anwen-anyi
Copy link
Contributor

anwen-anyi commented Feb 18, 2023 •
edited
Loading

不过好像还是一样的问题QAQ 我用的这个 期待其他人测试试试看 alist-org/alist/actions/runs/4210480253

还是 The system cannot find the path specified. ？

是的~ build时间也是对的~软件没下错

@kdxcxs
Copy link
Contributor Author

kdxcxs commented Feb 18, 2023 •
edited
Loading

不过好像还是一样的问题QAQ 我用的这个 期待其他人测试试试看 alist-org/alist/actions/runs/4210480253

还是 The system cannot find the path specified. ？

是的~ build时间也是对的~软件没下错

试试 cmd 直接运行不用 mingw 呢？

@anwen-anyi
Copy link
Contributor

anwen-anyi commented Feb 18, 2023 •
edited
Loading

不过好像还是一样的问题QAQ 我用的这个 期待其他人测试试试看 alist-org/alist/actions/runs/4210480253

还是 The system cannot find the path specified. ？

是的~ build时间也是对的~软件没下错 ![]

试试 cmd 直接运行不用 mingw 呢？

~~都一样的结果 cmd powershell 都试了一下，在你的设备商可以下载然后上传么

现在是不是改了？下载好了如果上传不了就给你删了把文件？

Windows 7

在Alist后台提示解析不了（解析时间似乎边长了），然后不会给你删除文件

Windows 10

能解析 能下载，
然后就是最后 提示 The system cannot find the path specified. ，同时 qbittorrent 下载的文件也被删除了

@anwen-anyi
Copy link
Contributor

powershell

powershell

cmd

cmd

@kdxcxs
Copy link
Contributor Author

kdxcxs commented Feb 18, 2023

我用 action 里面那个 build 在 C 盘试了一下，同样的种子我这边测试是没问题的

image.png

你的 qb 版本是多少？

@anwen-anyi
Copy link
Contributor

我用 action 里面那个 build 在 C 盘试了一下，同样的种子我这边测试是没问题的

你的 qb 版本是多少？

Windows10 4.5.0.10算新了吗？ Windows7是4.3.8.10

@anwen-anyi
Copy link
Contributor

QAQ 反转了.....难道是我下的绿色版或者版本低的问题吗？
刚刚我把Windows 7的更新了一下 去 https://github.com/c0re100/qBittorrent-Enhanced-Edition/releases/tag/release-4.5.1.10
下载了一个 直装版 然后可以用了..

@kdxcxs
Copy link
Contributor Author

kdxcxs commented Feb 18, 2023

QAQ 反转了.....难道是我下的绿色版或者版本低的问题吗？ 刚刚我把Windows 7的更新了一下 去 c0re100/qBittorrent-Enhanced-Edition@release-4.5.1.10 (release) 下载了一个 直装版 然后可以用了..

可能就是那个 qb 的问题，我用原版调试的，可能那些版本修改了一些东西或者设置在删除任务的时候把文件一起删除了吧。不过还好你总算用上了

@anwen-anyi
Copy link
Contributor

QAQ 反转了.....难道是我下的绿色版或者版本低的问题吗？ 刚刚我把Windows 7的更新了一下 去 c0re100/qBittorrent-Enhanced-Edition@release-4.5.1.10 (release) 下载了一个 直装版 然后可以用了..

可能就是那个 qb 的问题，我用原版调试的，可能那些版本修改了一些东西或者设置在删除任务的时候把文件一起删除了吧。不过还好你总算用上了

哈哈 麻烦和辛苦你了~ 不过也算是长记性了 可以把这俩记下来 记在小本本上面 供后人参考

@xhofe xhofe merged commit 3c7512f into AlistGo:main Feb 18, 2023
xhofe added a commit that referenced this pull request Feb 23, 2023
@xhofe
Squashed commit of the following:
0528ced 
commit de3ea82
Author: Andy Hsu <[email protected]>
Date:   Wed Feb 22 21:27:08 2023 +0800

    ci: add closeComment for stale

commit 268ba3d
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Wed Feb 22 21:24:35 2023 +0800

    fix(deps): update module github.com/gin-gonic/gin to v1.9.0 [skip ci] (#3551)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 309d655
Author: GodFinal <[email protected]>
Date:   Wed Feb 22 21:19:42 2023 +0800

    feat(local): add thumbnail for video with ffmpeg (#3556)

    * feat(local): add ffmpeg

    * fix: missed `+`

    ---------

    Co-authored-by: Andy Hsu <[email protected]>

commit c08fdfc
Author: Andy Hsu <[email protected]>
Date:   Wed Feb 22 20:20:28 2023 +0800

    fix: missed assignment [skip ci]

commit 1b28e6a
Author: Andy Hsu <[email protected]>
Date:   Wed Feb 22 20:07:18 2023 +0800

    ci: replace issues-helper with stale for inactive check

commit 8655e33
Author: Andy Hsu <[email protected]>
Date:   Tue Feb 21 19:57:50 2023 +0800

    fix: incorrect api if not set site_url (6c2f348)

commit 50579fe
Author: Andy Hsu <[email protected]>
Date:   Tue Feb 21 19:45:09 2023 +0800

    fix: cancel api replace to avoid missing host

commit e39299b
Author: Andy Hsu <[email protected]>
Date:   Tue Feb 21 17:45:15 2023 +0800

    fix(local): missed type of `MkdirPerm` (923937b)

commit d1ab244
Author: kdxcxs <[email protected]>
Date:   Tue Feb 21 16:45:41 2023 +0800

    feat(qbittorrent): delete tags when deleting qbittorrent tasks (#3546)

    * feat & refactor(qbittorrent/client): support `deleteFiles` arg for `Client.Delete()` method

    * feat(qbittorrent/client): also delete tags in `Client.Delete()`

commit 658cf36
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Tue Feb 21 16:43:37 2023 +0800

    fix(deps): update github.com/t3rm1n4l/go-mega digest to b87ebf5 (#3539)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit fd36ce5
Author: Andy Hsu <[email protected]>
Date:   Tue Feb 21 16:19:46 2023 +0800

    fix(onedrive): either id or path in parentReference must be specified (close #3028)

commit 95b3b87
Author: Andy Hsu <[email protected]>
Date:   Mon Feb 20 16:57:52 2023 +0800

    feat(sftp): support range header

commit 0d07d81
Author: Andy Hsu <[email protected]>
Date:   Mon Feb 20 16:46:38 2023 +0800

    feat(smb): support range header (close #3192)

commit 923937b
Author: Andy Hsu <[email protected]>
Date:   Mon Feb 20 16:20:36 2023 +0800

    feat(local): custom mkdir perm (close #3196)

commit 0949219
Author: Andy Hsu <[email protected]>
Date:   Mon Feb 20 16:15:52 2023 +0800

    fix(alist_v3): api error pass (close #3326)

commit 40b26a8
Author: Andy Hsu <[email protected]>
Date:   Mon Feb 20 16:08:10 2023 +0800

    fix!: change default epub viewer (close #3519)

commit 4293a0b
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Feb 20 16:06:35 2023 +0800

    fix(deps): update module github.com/golang-jwt/jwt/v4 to v4.5.0 [skip ci] (#3525)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 6c2f348
Author: Andy Hsu <[email protected]>
Date:   Sat Feb 18 19:03:07 2023 +0800

    fix!: reverse proxy to sub-directory (#3483)

    from this commit, if you want reverse proxy to sub-directory like `alist` with `nginx`, you need config:

    ```nginx
    location /alist/ {
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_set_header Host $http_host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header Range $http_range;
        proxy_set_header If-Range $http_if_range;
        proxy_redirect off;
        proxy_pass http://127.0.0.1:5244/alist/;
        # the max size of file to upload
        client_max_body_size 20000m;
    }
    ```

commit 3c7512f
Author: kdxcxs <[email protected]>
Date:   Sat Feb 18 18:54:51 2023 +0800

    fix(qbittorrent): fix two file transferring related bugs [skip ci] (#3501)

    * fix(qbittorrent): delete qbittorrent task before transferring

    * fix(qbittorrent): parse the path correctly when the torrent contains folders

commit 84219d3
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Sat Feb 18 18:13:41 2023 +0800

    fix(deps): update module gorm.io/driver/mysql to v1.4.7 [skip ci] (#3495)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 05d3727
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Sat Feb 18 18:13:22 2023 +0800

    fix(deps): update module golang.org/x/image to v0.5.0 [security skip ci] (#3489)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit ee77c3b
Author: 仝华帅 <[email protected]>
Date:   Sat Feb 18 17:53:11 2023 +0800

    fix: friendly tip for initial logging in [skip ci] (#3406)

    * refactor: friendly tip for initial logging in

    * fix CodeFactor issue

    more info pls refer to: https://segmentfault.com/a/1190000043031147

commit fcaf485
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Sat Feb 18 17:52:03 2023 +0800

    fix(deps): update module gorm.io/driver/postgres to v1.4.8 [skip ci] (#3496)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit bd83469
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Sat Feb 18 17:51:33 2023 +0800

    fix(deps): update module golang.org/x/net to v0.7.0 [security skip ci] (#3502)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 90f111b
Author: BaimoQilin <[email protected]>
Date:   Sat Feb 18 17:50:42 2023 +0800

    docs: translate title [skip ci] (#3498)

    * Update README_cn.md

    * Update README_cn.md

    ---------

    Co-authored-by: Andy Hsu <[email protected]>

commit 7d1034c
Author: foxxorcat <[email protected]>
Date:   Thu Feb 16 22:12:19 2023 +0800

    fix(aliyundrive): error occurred when running multiple instances at the same time (#3448)

    * fix(aliyundrive):an error occurred when running multiple instances at the same time

    * Update util.go

    fix(aliyunpan):clear retry count

commit 236c171
Author: Andy Hsu <[email protected]>
Date:   Thu Feb 16 22:09:33 2023 +0800

    fix(123): adapt new file list api (close #3464)

commit 6ee4c10
Author: Andy Hsu <[email protected]>
Date:   Thu Feb 16 21:37:20 2023 +0800

    chore(onedrive)!: change default redirect_uri [skip ci]

commit 3798634
Author: AkashiCoin <[email protected]>
Date:   Thu Feb 16 15:42:11 2023 +0800

    fix(pikpak_share): change media url to content url (close #3273) (#3441)
varg1714 pushed a commit to varg1714/alist that referenced this pull request Feb 26, 2023
@kdxcxs @walleoo
fix(qbittorrent): fix two file transferring related bugs [skip ci] (A…
82e8d67 
…listGo#3501)

* fix(qbittorrent): delete qbittorrent task before transferring

* fix(qbittorrent): parse the path correctly when the torrent contains folders
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kdxcxs @anwen-anyi @xhofe