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

Adding timestamp to filename #23

Closed
rfink opened this issue May 30, 2014 · 0 comments · Fixed by #24
Closed

Adding timestamp to filename #23

rfink opened this issue May 30, 2014 · 0 comments · Fixed by #24

Comments

@rfink
Copy link
Contributor

rfink commented May 30, 2014

This may not be the direction you guys are shooting for, so feel free to deny the pull request and close this out, although we had to modify this on our copy of rollbar.php to work in our environment. The current file name for the rollbar agent log is generated based on the process id of the current php process. In a high transaction environment, these could very well overlap. I'm proposing and adding a pull request in the new few moments that appends the timestamp to the process id to make sure a unique file name is generated. Thanks in advance for your consideration.

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 a pull request may close this issue.

1 participant