Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace hardcoded "./StORMlog.txt" with custom path #7

Open
neoneye opened this issue Jun 10, 2018 · 1 comment
Open

Replace hardcoded "./StORMlog.txt" with custom path #7

neoneye opened this issue Jun 10, 2018 · 1 comment

Comments

@neoneye
Copy link
Contributor

neoneye commented Jun 10, 2018

The logFile parameter is set to "./StORMlog.txt", several places. Looks like this:

if StORMdebug { LogFile.info("Error msg: \(errorMsg)", logFile: "./StORMlog.txt") }
if StORMdebug { LogFile.info("Postgres conn state: ok", logFile: "./StORMlog.txt") }
LogFile.error("Error msg: \(error)", logFile: "./StORMlog.txt")

This polutes the current working directory. I prefer logfiles gets outputted to the log dir.

Perhaps use a shared logging class.

Same issue with Mysql + Sqlite repositories.

@Zollerboy1
Copy link

+1 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

No branches or pull requests

2 participants