Possible permissions issues with /status #12

Closed
opened 6 years ago by ajohnson · 3 comments
Owner

Example output:

Binary file /tmp/.stat matches

Online players:
- - - - - - - - - - - - -

Further testing shows this is highly dependent on the server system, but a workaround to avoid it should be easy.

Example output: ``` Binary file /tmp/.stat matches Online players: - - - - - - - - - - - - - ``` Further testing shows this is highly dependent on the server system, but a workaround to avoid it should be easy.
ajohnson self-assigned this 6 years ago
Poster
Owner

Furthermore, .stat is a potentially dangerously generic file name to be used in /tmp.

Furthermore, .stat is a potentially dangerously generic file name to be used in /tmp.
ajohnson started working 6 years ago
Poster
Owner

Additionally, multiple bots on one server could feasibly interfere with each other using a static file name. I will be making the tmpdir a configurable setting instead.

Additionally, multiple bots on one server could feasibly interfere with each other using a static file name. I will be making the tmpdir a configurable setting instead.
ajohnson stopped working 6 years ago
1h 11min 23s
Poster
Owner

Fixed:
69381daf66

Fixed: https://git.skyfall.tech/skyfall/empyrion-bot/commit/69381daf662708147549655494301ffdcbaf0c1b
ajohnson closed this issue 6 years ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
1 Participants
Notifications
Total Time Spent: 1 hour 11 minutes
ajohnson
1 hour 11 minutes
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: skyfall/empyrion-bot#12
Loading…
There is no content yet.