Refactor receiver webhook client to support custom dialers
- Introduced client options in HTTP client creation similar to upstream.
- Added support for custom
DialContextin webhook client. - Enhanced integration tests to validate custom dialer behavior.
Part of a series of PRs for full support. This PR lacks support for: mqtt, sns, slack, and prometheus-alertmanager. It also leaves more in-depth client refactoring to followups.
Most changes to tests are to facilitate the new factory_test that verifies a provided custom dialer blocking requests works as expected. Various issues with the "valid" testing configs for integrations were causing them to fail for other reasons before they hit the dialer.
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.
I am not sure I understand where the hard-coded timeouts came from. Also, I am worried about coordination of hard-coded and configured timeouts. Other than that, LGTM