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

Ignoring production configs #83

Merged
merged 4 commits into from
Nov 28, 2013
Merged

Conversation

peelman
Copy link

@peelman peelman commented Nov 28, 2013

After the fourth time I clobbered them today i figured I should take a few minutes and do this...

snipe added a commit that referenced this pull request Nov 28, 2013
@snipe snipe merged commit 39cc0c6 into snipe:develop Nov 28, 2013
@snipe
Copy link
Owner

snipe commented Nov 28, 2013

FYI, I'm going to change mv to cp in the read me, and will probably also change the example names to blah.example.php from blah.php.example. Since apache won't parse a .example file, if that config (recently updated by the user with real db creds) were to accidentally end up in a webroot somewhere, it would expose sensitive data - versus ending it with php, where it just gets parsed and no data leaked.

@peelman
Copy link
Author

peelman commented Nov 28, 2013

👍

@peelman peelman deleted the IgorningProductionConfigs branch November 28, 2013 15:26
@veen1981 veen1981 mentioned this pull request Jul 7, 2016
2 tasks
This was referenced Jul 19, 2016
@dooley1023 dooley1023 mentioned this pull request Feb 2, 2017
2 tasks
@katecruser katecruser mentioned this pull request Feb 8, 2017
2 tasks
@Mike1704 Mike1704 mentioned this pull request Mar 3, 2017
@jross-nu jross-nu mentioned this pull request Sep 13, 2019
2 tasks
@anees78692 anees78692 mentioned this pull request Nov 29, 2019
@LeeSharman LeeSharman mentioned this pull request Feb 26, 2021
@ecmcidev ecmcidev mentioned this pull request Apr 6, 2022
2 tasks
@cookierun22 cookierun22 mentioned this pull request Jan 28, 2023
2 tasks
@Bjufen Bjufen mentioned this pull request Feb 9, 2023
2 tasks
@jayavman jayavman mentioned this pull request Sep 2, 2023
2 tasks
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 this pull request may close these issues.

2 participants