-
Notifications
You must be signed in to change notification settings - Fork 947
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
Leading hyphen in log file names #178
Labels
Comments
We would accept a pull request here, as well as for your idea for |
Pull request sent for first part of above, https://github.com/bvmou/forever/commit/65f2f5d611475c05fc51738bf64269926d1f2daa
|
This was referenced Jan 6, 2020
This was referenced Apr 9, 2022
This was referenced Apr 16, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Not sure if this is yet a violation of posix but probably not the right behavior (http://www.dwheeler.com/essays/fixing-unix-linux-filenames.html#dashes http://www.opengroup.org/austin/docs/austin_224.txt ).
Am not competent to suggest best fix but easy first pass that preserves approximately equivalent randomness is just:
// lib/forever.js line 325
options.uid = options.uid || utile.randomString(4).replace(/^\-/, 'H')
If you are in touch with the utile developer might be nice to have optional chars argument for cases like this.
The text was updated successfully, but these errors were encountered: