Skip to content

FIX: Fixes #1074 By passing 4th param to captureMessage().#1075

Closed
phptek wants to merge 1 commit into
Seldaek:masterfrom
phptek:issue/1074
Closed

FIX: Fixes #1074 By passing 4th param to captureMessage().#1075
phptek wants to merge 1 commit into
Seldaek:masterfrom
phptek:issue/1074

Conversation

@phptek
Copy link
Copy Markdown

@phptek phptek commented Nov 2, 2017

A 4th param is now passed to to RavenHandler::captureMessage()

phptek pushed a commit to phptek/silverstripe-sentry that referenced this pull request Nov 2, 2017
phptek pushed a commit to phptek/silverstripe-sentry that referenced this pull request Nov 2, 2017
- Bumped composer reqs
- Re-arranged things under a "src" dir
- Fixes inability to send stacktrace. (Relies on upstream PR: Seldaek/monolog#1075)
- Improved README and docs
- Modified travis config
@phptek
Copy link
Copy Markdown
Author

phptek commented Jan 23, 2018

Bump

@Seldaek
Copy link
Copy Markdown
Owner

Seldaek commented Jun 8, 2018

The right way to capture the stack is to pass an exception instance in the log context's exception key (as defined in PSR-3), then Raven's captureException is used, which captures the stack.

@Seldaek Seldaek closed this Jun 8, 2018
charlieqhicks added a commit to charlieqhicks/silverstripe-sentry that referenced this pull request Feb 10, 2022
- Bumped composer reqs
- Re-arranged things under a "src" dir
- Fixes inability to send stacktrace. (Relies on upstream PR: Seldaek/monolog#1075)
- Improved README and docs
- Modified travis config
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