secondlife / secondlife/viewer
Portable viewer should stores sensitive data in viewer's folder
Open
A pull request for this has already been merged.
- #6299 by @akleshchev — merged
- Dominant language
- C++
- Stars
- 299
- Forks
- 146
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 88
Description
Portable viewer should stores sensitive data in viewer's folder instead of a temporary PC.
Test plan:
- Unpack and launch portable viewer
- Make some changes to settings
- Stop viewer and launch again to confirm settings saved
- Check viewer's root folder (the one .portable and Updater.exe are in) and remove UserData
- Launch viewer again
- Confirm removing UserData dropped the settings
Expected:
- Settings to persist with UserData in viewer's folder.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing merged pull request #6299 and the portable viewer startup and settings-persistence path; the issue names no files or automated tests. Reproduce the listed portable-viewer test plan, then verify that settings are stored in UserData under the viewer's root folder and disappear when that folder is removed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- desktop
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100