Code

Added exception monitoring client#243

Code

Merged

asgrim merged 67 commits intomasterscoutapp/scout-apm-php:masterfrom exception-monitoring-clientscoutapp/scout-apm-php:exception-monitoring-clientCopy head branch name to clipboardJan 11, 2022

+3,738-298

Commits

Commits on Jan 11, 2022

• Added ErrorEvent to capture details about a \Throwable in the expected Scout Errors payload format

  asgrim

  committed
  0460330

• Added utility class to gzip the payload

  asgrim

  committed
  0cd0aa2

• Added basic implementation of ErrorReportingClient using Guzzle

  asgrim

  committed
  979f79d

• Add accessor for request ID to Agent

  asgrim

  committed
  df578ac

• Added ErrorHandling class with initial integration test

  asgrim

  committed
  a272508

• Updated expected shape for traces and request parameters

  asgrim

  committed
  7605250

• Removed requestId method from Agent

  asgrim

  committed
  86efe61

• Instantiate default ScoutErrorHandling implementation in Agent::fromConfig

  asgrim

  committed
  77188d5

• Make RequestId optional in the error reporting to ensure it always gets added

  asgrim

  committed
  32847d9

• Extract application root finding into FindApplicationRoot

  asgrim

  committed
  d1a6910

• Format trace in ErrorEvent to expected format for Scout

  asgrim

  committed
  a83f4e9

• Added configuration options from Python agent to configure error reporting (off by default)

  asgrim

  committed
  6120433

• Added some basic debug logging to ScoutErrorHandling

  asgrim

  committed
  e74ed05

• Added uncaught exception test for integration tests

  asgrim

  committed
  a9f02dd

• Use Request generated path in ErrorEvent creation

  asgrim

  committed
  1796d7a

• Extract filtering of given parameters into a separate helper

  asgrim

  committed
  b832570

• Pass session and env variables through filtering for ErrorEvent

  asgrim

  committed
  1a1f537

• Added real hostname from Config or gethostname() using consistent helper

  asgrim

  committed
  0ea331a

• Offload superglobal retrieval to static helper

  asgrim

  committed
  cd71d9e

• Extract RootPackageGitSha function to re-usable static helper

  asgrim

  committed
  0e99b7c

• Build full request URI for ErrorEvent

  asgrim

  committed
  a19b39e

• Filter and flatten request parameters into ErrorEvent better

  asgrim

  committed
  eec01e6

• Pass tags from Request into ErrorEvent context

  asgrim

  committed
  e739b84

• Push controller or job name into request_components for ErrorEvent

  asgrim

  committed
  3788206

• Added configuration option for errors_filtered_params

  asgrim

  committed
  7441783

• Updated scout-apm-php to depend on php-http/discovery to find PSR-17 and PSR-18 implementations

  asgrim

  committed
  c9c8032

• Added incomplete unit tests for changeset

  asgrim

  committed
  80766f1

• Move superglobals into an injectable service to separate static instantiation

  asgrim

  committed
  6d70112

• Refactored DetermineHostname to be an injectable service

  asgrim

  committed
  d73fb85

• Refactored FindRootPackageGitSha to injectable service

  asgrim

  committed
  0f7571b

• Define interface for FindApplicationRoot service

  asgrim

  committed
  add55b3

• Make FindRequestHeaders an injectable service

  asgrim

  committed
  32dc947

• Define an interface for LocateFileOrFolder

  asgrim

  committed
  043b21f

• Added testcases for Request->requestPath

  asgrim

  committed
  427f32f

• Added tests for error handling and new helpers

  asgrim

  committed
  2c23173

• Removed todo for ERROR_TYPES_TO_CATCH

  asgrim

  committed
  089b5c5

• Updated Psalm baseline for version update

  asgrim

  committed
  88a4788

• Added recordThrowable method to Agent

  asgrim

  committed
  f294766

• Added new magic facade method to ScoutApm Facade for Laravel integration

  asgrim

  committed
  7ed42f3

• Added changelog entry

  asgrim

  committed
  26b9e84

• Depend on Scouts internal TestLogger implementation not Psr\Log\Test

  asgrim

  committed
  f7a35ac

• Adjust number of paths skipped in LocateFileOrFolderUsingFilesystem since moving to sub-directory

  asgrim

  committed
  7c38ccd

• Dependency fixes for GH Actions builds

  asgrim

  committed
  18f1013

• Enabled Symfony exception listener

  asgrim

  committed
  aeff9bb

• Updated ErrorReportingClient to expect an array of ErrorEvents for forward-compatibility if batches are added

  asgrim

  committed
  c52461c

• Made ErrorReporting an optional component which raises information warning if error_reporting=true and an HTTP client is not found.

  asgrim

  committed
  a2a7fd5

• PHP 7.1 requires older Guzzle so install that for dev deps

  asgrim

  committed
  72e2961

• Provide log file written in testUncaughtErrorsAreCapturedAndSentToScout to failure output

  asgrim

  committed
  0643660

• Added E2E test for Laravel error reporting component

  asgrim

  committed
  5e41a59

• Clarification in CHANGELOG for LocateFileOrFilder BC break

  asgrim

  committed
  4fa3a04

• Elevate dependency on FindRequestHeadersUsingServerGlobal to named constructor

  asgrim

  committed
  ea65fe6

• RequireValidFilteredParameters is now internal API

  asgrim

  committed
  f523e81

• Renamed ErrorEventJsonStructure type for clarity

  asgrim

  committed
  e4126a0

• Exception must be class-string of Throwable

  asgrim

  committed
  9c16787

• Extract logic for determining port and host for current URL to be clearer

  asgrim

  committed
  68f4664

• Check that controller/action keys are defined in ErrorEvent

  asgrim

  committed
  1068095

• Doc blocks provided for ErrorHandling interface

  asgrim

  committed
  23a1d12

• Inject Superglobals into Request instead of global state load

  asgrim

  committed
  f9f08ae

• Use array_values(..)[0] instead of reset(..)

  asgrim

  committed
  4e0da3c

• Make it explicit that \Scoutapm\Helper\FilterParameters::flattenedForUriReportingConfiguration supports positive-int|0

  asgrim

  committed
  71f817c

• Memoize application root in FindApplicationRootWithConfigOverride

  asgrim

  committed
  42b25c6

• Use Superglobals instead of getenv for Heroku override

  asgrim

  committed
  a5c729c

• Throw exception when invalid config value used for $filteredParameter

  asgrim

  committed
  5948dd1

• Rename errorsUrl method and memoization property for clarity

  asgrim

  committed
  727ce10

• Added specific log message when HTTP 429 returned from error reporting URL

  asgrim

  committed
  000bdce

• Silence deprecated errors in PHPUnit

  asgrim

  committed
  bb41829

• Move registerListeners call to the default ErrorHandlingDiscoveryFactory

  asgrim

  committed
  e4167ee