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

Prbranch #45

Merged
merged 3 commits into from
Jan 13, 2024
Merged

Prbranch #45

merged 3 commits into from
Jan 13, 2024

Conversation

lauralex
Copy link
Owner

@lauralex lauralex commented Jan 13, 2024

Related PR: #44

  • Reduce probability of temp file permission errors

It less likely we will encounter system permission changes that affect the app's read/write access by using the system-determined temp path instead of always using Window's temp directory.

  • Add missing trailing path separator

  • Privilege escalation

Signed-off-by: Alessandro Bellia casilda.bell85@gmail.com


Signed-off-by: Alessandro Bellia casilda.bell85@gmail.com
Co-authored-by: CJ Davis code@cjdavis.me

Crevax and others added 3 commits January 11, 2024 17:18
It less likely we will encounter system permission changes that affect the app's read/write access by using the system-determined temp path instead of always using Window's temp directory.
Signed-off-by: Alessandro Bellia <casilda.bell85@gmail.com>
@lauralex lauralex merged commit 4df6f84 into master Jan 13, 2024
@lauralex lauralex deleted the prbranch branch January 13, 2024 17:38
lauralex added a commit that referenced this pull request Jan 13, 2024
* Reduce probability of temp file permission errors

It less likely we will encounter system permission changes that affect the app's read/write access by using the system-determined temp path instead of always using Window's temp directory.

* Add missing trailing path separator

* Privilege escalation

Signed-off-by: Alessandro Bellia <casilda.bell85@gmail.com>

---------

Signed-off-by: Alessandro Bellia <casilda.bell85@gmail.com>
Co-authored-by: CJ Davis <code@cjdavis.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants