Skip to content
Snippets Groups Projects

webhook: Fix running with no proxies

Merged Emanuele Aina requested to merge wip/em-fix-the-fix-to-the-proxy-fix into master

The fix to the proxy fix did break running with no proxies as ProxyFix doesn't like being passed 0 as the number of proxies.

For this reason the unit tests were also broken. Sigh.

Merge request reports

Pipeline #4150 passed

Pipeline passed for f99a502f on wip/em-fix-the-fix-to-the-proxy-fix

Approval is optional

Merged by Luis AraujoLuis Araujo 5 years ago (Jul 10, 2019 7:05am UTC)

Merge details

  • Changes merged into master with f99a502f.
  • Deleted the source branch.

Pipeline #4154 passed

Pipeline passed for f99a502f on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading