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.

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

Uninstall don't delete %AppData% #1864

Open
RubenKelevra opened this issue Jun 25, 2021 · 3 comments
Open

Uninstall don't delete %AppData% #1864

RubenKelevra opened this issue Jun 25, 2021 · 3 comments
Labels
area/windows Windows effort/hours Estimated to take one or several hours good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked

Comments

@RubenKelevra
Copy link
Contributor

  • OS: Windows
  • Version of IPFS Desktop 0.16.0

Describe the bug
The following folders are created by IPFS-Desktop while it's being used by the user:

%AppData%\Roaming\IPFS Desktop
%USERPROFILE%\.ipfs
%USERPROFILE%\.ipfs-desktop

None of which will be removed when IPFS-Desktop will be uninstalled.

Following the principle of the least surprise this should be the case, as this is the warn message Windows shows when uninstalling an application:

ipfs_desktop_screenshot_bugreport_no_data_cleanup_on_uninstall_uninstall_message

To Reproduce
Steps to reproduce the behavior:

  1. Use IPFS Desktop
  2. Uninstall it via the Settings -> Apps
  3. Complete the uninstall procedure
  4. Look at the quoted paths, which still exist

Expected behavior
All data associated with IPFS Desktop should be removed when uninstalling.

@RubenKelevra RubenKelevra added the need/triage Needs initial labeling and prioritization label Jun 25, 2021
@lidel
Copy link
Member

lidel commented Jul 2, 2021

  • 💔 removal of user data (IPFS_PATH, aka .ipfs) as the default is a bit controversial: we would not do this unless we have a separate, explicit opt-in from the user. Does not seem to be doable with electron-builder tho.

  • 💚 ❓ removal of %AppData% (as long it does not include .ipfs) sounds sensible

    • I think flipping deleteAppDataOnUninstall in electron-builder.yml would do the trick, but actual behavior needs to be verified by someone using MS Windows – PR welcome!

@lidel lidel added the area/windows Windows label Jul 2, 2021
@lidel lidel changed the title Uninstall don't delete any user-data Uninstall don't delete %AppData% Jul 2, 2021
@lidel lidel added effort/hours Estimated to take one or several hours good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked and removed need/triage Needs initial labeling and prioritization labels Jul 2, 2021
@RubenKelevra
Copy link
Contributor Author

Some programs do ask for "settings and config files" or something like this on uninstallation. But to be honest the default is to expect that everything will be gone after deinstallation.

Since there are no "files and folders" somewhere for the users to see the default expectation is, that everything stored "within" the program should be gone as well on deinstallation.

@RubenKelevra
Copy link
Contributor Author

  • I think flipping deleteAppDataOnUninstall in electron-builder.yml would do the trick, but actual behavior needs to be verified by someone using MS Windows – PR welcome!

This would lead to the result I am proposing when this solution is implemented:

#1656 (comment)

So the data stored in ipfs would then be gone as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/windows Windows effort/hours Estimated to take one or several hours good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked
Projects
No open projects
Status: Needs Grooming
Development

No branches or pull requests

2 participants