X Tutup
The Wayback Machine - https://web.archive.org/web/20210210074106/https://github.com/github/gitignore/pull/3230
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

Add crash.log to Packer.gitignore #3230

Merged
merged 1 commit into from Nov 8, 2019
Merged

Conversation

@sean-nixon
Copy link
Contributor

@sean-nixon sean-nixon commented Nov 8, 2019

Reasons for making this change:
If Packer crashes, it writes the panic output to the file crash.log in the current directory. This file should not be tracked in version control as it has the potential to contain sensitive information and is only useful for debugging. This behavior is the same as Terraform, which already includes crash.log in its .gitignore template

Links to documentation supporting these rule changes:

I was unable to find official documentation on this behavior, but it's pretty clear in panic output string in the source code here: https://github.com/hashicorp/packer/blob/master/panic.go

If packer crashes, it write logs to the file "crash.log" in the current directory. This file should not be tracked in version control.
Copy link
Member

@shiftkey shiftkey left a comment

@sean-nixon thanks!

@shiftkey shiftkey merged commit 766cf0c into github:master Nov 8, 2019
tokuhira added a commit to tokuhira/gitignore that referenced this pull request Dec 16, 2019
If packer crashes, it write logs to the file "crash.log" in the current directory. This file should not be tracked in version control.
EffingFancy added a commit to EffingFancy/gitignore that referenced this pull request Feb 23, 2020
If packer crashes, it write logs to the file "crash.log" in the current directory. This file should not be tracked in version control.
r2pgl added a commit to r2pgl/gitignore that referenced this pull request Mar 15, 2020
If packer crashes, it write logs to the file "crash.log" in the current directory. This file should not be tracked in version control.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
X Tutup