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

Catch access denied #684

Closed
1-alex98 opened this issue Jan 9, 2018 · 5 comments · Fixed by #686
Closed

Catch access denied #684

1-alex98 opened this issue Jan 9, 2018 · 5 comments · Fixed by #686
Assignees
@1-alex98
Labels
enhancement

Comments

@1-alex98
Copy link
Member

1-alex98 commented Jan 9, 2018

If the user starts the client as admin and later on starts it as normal user he might get access denied exceptions because he has not all permissions for the game files....
@1-alex98 1-alex98 self-assigned this Jan 9, 2018
@1-alex98 1-alex98 added this to the v0.8.0-beta milestone Jan 9, 2018
@1-alex98
Copy link
Member Author

Thats not what we should do...
There is a better solution:
The error is caused by the client being unable to delete the stylesheet cache, if we delted that files on close up, no problems will occur. As on close up we have the same permission as on creation.

1-alex98 added a commit that referenced this issue Jan 10, 2018
@1-alex98
Delete cash directory on close
15a15c8 
Deletes cash directory on close and start up
Fixes #684
@ghost ghost added the in progress label Jan 10, 2018
@1-alex98 1-alex98 mentioned this issue Jan 10, 2018
Merged
@micheljung
Copy link
Member

It's not just the cache directory, the same problem occurs with the game files and we shouldn't just delete those when the client closes.

@1-alex98
Copy link
Member Author

Ok then I'll look at this as well ;)

@micheljung micheljung modified the milestones: v0.8.0-beta, v0.8.1-beta Jan 19, 2018
micheljung pushed a commit that referenced this issue Feb 3, 2018
@1-alex98 @micheljung
Delete cash directory on close
90611e7 
Deletes cash directory on close and start up
Fixes #684
@ghost ghost removed the in progress label Feb 3, 2018
@micheljung micheljung modified the milestones: v0.8.1-beta, v0.9.0-beta May 1, 2018
@micheljung
Copy link
Member

This not yet fixed

@micheljung micheljung reopened this May 24, 2018
@micheljung micheljung removed this from the v0.9.0-beta milestone May 24, 2018
@Brutus5000
Copy link
Member

User gets a warning now etc.

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

@1-alex98 1-alex98

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Delete cache directory on close FAForever/downlords-faf-client
3 participants
@micheljung @Brutus5000 @1-alex98