artin/sourcegraph
1
0
Fork 0
mirror of https://github.com/sourcegraph/sourcegraph.git synced 2026-02-06 18:31:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

webhooks: Remove TODO (#43662)

Browse Source 
We've already done this, notice that we wrap `webhookHandler` with `webhookMiddleware.Logger`.
This commit is contained in:
Ryan Slade 2022-10-31 10:55:16 +01:00 committed by GitHub
parent b5a40fdbe7
commit cb52e51164
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
cmd/frontend/internal/httpapi/httpapi.go
View File

@ -98,7 +98,6 @@ func NewHandler(
ghSync.Register(&gh)
// 🚨 SECURITY: This handler implements its own secret-based auth
// TODO: Integrate with webhookMiddleware.Logger
webhookHandler := webhooks.NewHandler(logger, db, &gh)
m.Get(apirouter.Webhooks).Handler(trace.Route(webhookMiddleware.Logger(webhookHandler)))