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

Change, It's now possible to configure number of inner exceptions to be sent #14

Merged
merged 2 commits into from Nov 14, 2018

Conversation

@ChiliMonanta
Copy link

@ChiliMonanta ChiliMonanta commented Dec 19, 2017

Change, It's now possible to configure number of inner exceptions to be sent

Sometimes you need more than the default number of inner exceptions.
This is now possible to configurate.

Default value is not changed, it's still one.

…be sent

Sometimes you need more than the default number of inner exceptions.
This is now possible to configurate.

<numberOfInnerExceptions value="4"/>

Default value is not changed, it's still one.
@mostlyjason
Copy link

@mostlyjason mostlyjason commented Dec 19, 2017

Thanks for writing this up! It looks like a good addition and we appreciate the test cases as well. We'll need to run the tests and I'm not sure when we'll have time. Other users can grab this PR in the meantime to try it out.

@Katulus Katulus merged commit bf8b0ee into loggly:master Nov 14, 2018
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

3 participants
You can’t perform that action at this time.