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

Logger: can now report errors by other means than email #166

Closed
wants to merge 1 commit into from

Conversation

kravco
Copy link
Contributor

@kravco kravco commented May 8, 2016

My aim is to be able to report errors via text messages. But idea is to allow any other means for reporting, e.g. create issue in internal reporting system--the advantage would be, that it would contain all the necessary information for future reference.

Possibly in the future, email would lose it's special status because--in the end--it is just another reporting channel, no different from others, mentioned earlier.

If agreed, I suggest traditional slow transition, with:

  1. full backwards compatibility
  2. bc with triggering deprecated warning
  3. release with only new functionality

I'm ready to prepare these steps too.

@JanTvrdik
Copy link
Contributor

👎 I'd suggest using Monolog instead.

@dg dg force-pushed the master branch 2 times, most recently from 333e9f8 to 805f797 Compare May 17, 2016 15:23
@dg
Copy link
Member

dg commented May 17, 2016

Welcome back @kravco :-)

I agree that reporting via email is too simple. On the other hand, I do not know if it makes sense to develop logging part of Tracy, because it is not primarily a logging tool and there are better tools. Usually, when someone wants to use more sophisticated logging, he creates its own Logger, while the reportChannels are not enough complex solution.

@kravco
Copy link
Contributor Author

kravco commented May 17, 2016

Yeah, it's been a long time :-)

Ok, I understand. I can write my own logger for this.

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.

None yet

3 participants