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

pick some recently fixes #2524

Draft
wants to merge 5 commits into
base: release/v23
Choose a base branch
from
Draft

pick some recently fixes #2524

wants to merge 5 commits into from

Conversation

itsXuSt
Copy link
Contributor

@itsXuSt itsXuSt commented Dec 26, 2024

  • fix: [mount] passwd not clear when all unmounted.
  • refactor: improve network mount password handling
  • fix: refactoring the deletion logic for temporary passwords
  • fix: add dmask fmask when mount device
  • fix: Remove view mode settings when reset view mode
  • fix: save last visited url in filedialog.

@itsXuSt
fix: save last visited url in filedialog.
b5f2a3e 
- if directory is not setted when using FileDialog, use lastVisitedUrl
as default;

Log: as above.
@itsXuSt
fix: [mount] passwd not clear when all unmounted.
78858d7 
saved password should be cleared when all associated smb share
unmounted.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-275557.html
Change-Id: Ia2434bd4f2c47594aba24b6dbfa554cb324116a2
@itsXuSt
refactor: improve network mount password handling
a44a124 
- Refactor password saving logic for network mounts
- Change default password save mode to 'Never Save' instead of 'Save
Before Logout'
- Add temporary configuration file to track SMB password save mode
- Optimize password cleanup when unmounting SMB shares

This change improves security by making password saving more explicit
and
cleaning up credentials properly when unmounting network shares.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-279371.html
Change-Id: I09833084aba2f242d0e63502830dbe72e34846e7
@itsXuSt
fix: refactoring the deletion logic for temporary passwords
ba2c7e3 
- clear the password when the last smb of same host unmounted;
- only clear the password saved in `session` collection.

Log: fix samba password clean issue

Bug: https://pms.uniontech.com/bug-view-294555.html
Change-Id: I675584cff7907956d10b70889c0ecb71f5aaabbf
max-lvs
max-lvs previously approved these changes Dec 26, 2024
View reviewed changes
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见：

  1. 在application.cpp文件中，appAttributeTrigger函数中，continue语句后缺少分号，应添加分号以避免语法错误。
  2. 在devicemanager.cpp文件中，mountBlockDevAsync函数中，options变量在函数内部被修改，但未在函数外部使用，建议移除options变量的定义。
  3. 在devicemanager.cpp文件中，askForPasswdWhenMountNetworkDevice函数中，info.cancelled = true;后缺少return语句，应添加以避免逻辑错误。
  4. 在mountaskpassworddialog.cpp文件中，handleConnect函数中，loginObj.insert(kPasswdSaveMode, kSaveBeforeLogout);应改为loginObj.insert(kPasswdSaveMode, kNeverSave);以符合注释中的描述。
  5. 在filedialoghandle.cpp文件中，FileDialogHandle构造函数中，defaultUrl变量未在else分支中初始化，可能导致未定义行为。
  6. 在filedialog.cpp文件中，FileDialogPrivate构造函数中，lastVisitedDir变量未在else分支中初始化，可能导致未定义行为。
  7. 在filedialog.cpp文件中，FileDialog::cd函数中，d->lastVisitedDir = url;应在if语句块中，以避免在else分支中重复设置。
  8. 在filedialog.cpp文件中，FileDialog::initConnect函数中，connect语句中缺少using namespace，应添加以避免命名空间冲突。
  9. 在fileview.cpp文件中，FileView::onDefaultViewModeChanged函数中，if (newMode == d->currentViewMode)应改为if (newMode == d->currentViewMode)以避免逻辑错误。
  10. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_new_full函数的参数顺序错误，应将g_str_hash和g_str_equal参数位置互换。
  11. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  12. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  13. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  14. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  15. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  16. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  17. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  18. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  19. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  20. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  21. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  22. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  23. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  24. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  25. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  26. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  27. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  28. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  29. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  30. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  31. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  32. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  33. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  34. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  35. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  36. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  37. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  38. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  39. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  40. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  41. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  42. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  43. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  44. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  45. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  46. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  47. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  48. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  49. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  50. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  51. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  52. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  53. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  54. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  55. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  56. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  57. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  58. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  59. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  60. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  61. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  62. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  63. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  64. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  65. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  66. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  67. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  68. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  69. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  70. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  71. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  72. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  73. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  74. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  75. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  76. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  77. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  78. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  79. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  80. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  81. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  82. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  83. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession

@itsXuSt itsXuSt marked this pull request as draft December 26, 2024 12:10
@itsXuSt
fix: size not updated in admin mode.
f930656 
size update timer is not running in admin mode.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-278679.html
Change-Id: I5ae46ce6362656904e2ff5f74afa94d7bcdd45b7
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: itsXuSt

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

1 similar comment
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: itsXuSt

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@max-lvs max-lvs max-lvs left review comments

At least 1 approving review is required to merge this pull request.

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
@itsXuSt @deepin-ci-robot @max-lvs