Gitea: Webhook Authorization Header Returned in Plaintext via API

The ToHook() function in services/webhook/general.go decrypts the webhook's HeaderAuthorizationEncrypted field and returns the plaintext authorization header in the API response. Any repository admin can read the full plaintext value of webhook authorization headers (Bearer tokens, Basic auth credentials, API keys) set by other admins.

The authorization header is stored encrypted in the database using the server's SecretKey, but ToHook() decrypts it before serializing it into the API response — converting a write-only secret into a readable credential.

File: services/webhook/general.go:407-420

 1func ToHook(repoLink string, w *webhook_model.Webhook) (*api.Hook, error) {
 2    // ...
 3    authorizationHeader, err := w.HeaderAuthorization()  // DECRYPTS from DB
 4    if err != nil {
 5        return nil, err
 6    }
 7    return &api.Hook{
 8        // ...
 9        AuthorizationHeader: authorizationHeader,  // PLAINTEXT in response
10        // ...
11    }, nil
12}

Decryption function: models/webhook/webhook.go:209-216

 1func (w Webhook) HeaderAuthorization() (string, error) {
 2    if w.HeaderAuthorizationEncrypted == "" {
 3        return "", nil
 4    }
 5    return secret.DecryptSecret(setting.SecretKey, w.HeaderAuthorizationEncrypted)
 6}

All call ToHook():

• GET /api/v1/repos/{owner}/{repo}/hooks (requires repo admin)
• GET /api/v1/repos/{owner}/{repo}/hooks/{id} (requires repo admin)
• GET /api/v1/admin/hooks (requires site admin)
• GET /api/v1/orgs/{org}/hooks (requires org admin)
• GET /api/v1/user/hooks (requires authenticated user)

1. Admin A creates a webhook with a sensitive authorization header.
2. Admin B (different repo admin) lists webhooks via GET /api/v1/repos/{owner}/{repo}/hooks.
3. The API response includes the full plaintext authorization header set by Admin A.

• Cross-admin secret exposure on shared repositories
• Credential harvesting if a repo admin's Gitea token is stolen
• External service compromise via leaked Bearer tokens and API keys
• Undermines the intentional encryption-at-rest protection

The authorization header should be write-only. Return a masked/redacted version or a boolean has_authorization_header flag instead.

• Vulnerable function: services/webhook/general.go:392-425
• Decryption function: models/webhook/webhook.go:209-216
• Verified against commit 19f0169