-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Backport changes to v5.0 branch #20996
Open
sledgehammer999
wants to merge
32
commits into
qbittorrent:v5_0_x
Choose a base branch
from
sledgehammer999:backport_v5
base: v5_0_x
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+5,466
−5,018
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR restores the users previously used tab (Transfer, Search, RSS, etc.) when the WebUI is reloaded. PR qbittorrent#20705.
The cookie value can only contain ASCII characters.
PR qbittorrent#20913. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Chocobo1 <[email protected]>
Add "BT" (BitTorrent) to avoid confusion about which protocol it is referring to. Also its value doesn't need to be translated. PR qbittorrent#20897.
PR qbittorrent#20833. --------- Co-authored-by: Vladimir Golovnev <[email protected]> Co-authored-by: Chocobo1 <[email protected]>
A few code are revised because the type checker (mypy) doesn't allow changing types on a variable. PR qbittorrent#20935.
1. Previously unhandled connections will stay in pending state. It won't be closed until timeout happened. This may lead to wasting system resources. Now the (over-limit) connection is actively rejected. 2. When out-of-memory occurs here, reject the new connection instead of throwing exception and crash. 3. Also clean up some unused bits. PR qbittorrent#20961.
… of 0 Fixes the download progress calculation for folders with zero size. Previously, the progress would be Zero. Now, folders with zero size show 100% progress. PR qbittorrent#20567.
The canonical type for `size_string` is `str`. However numeric types are also accepted in order to accommodate poorly written plugins. PR qbittorrent#20976.
Previously `m_shutdownTimeout * 1000` was calculated in `int` and now it is `qint64`. PR qbittorrent#20982.
It does not affect the produced artifacts. The only difference is the tool itself won't depend on some specific OS image or library version. PR qbittorrent#20983.
glassez
approved these changes
Jun 25, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved generally, without taking into account possible problems in resolving the mentioned conflicts.
Chocobo1
approved these changes
Jun 27, 2024
PR #20990 should also be backported. |
And #20998 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Continuation of PR #20918 with the conflicts resolved.
There was only one commit that conflicted:
master version: cb90b67
v5 version: 4687b4e
Files in conflict: