fix(filtering): reduce verbose logging in hot paths
- commit
- 5e0ca2d5c6021eb7706ea045f259d4befdc4a308
- author
- bnema · · 2025-11-29 11:46 UTC
- parent
- 5ed8d0d
- signature
- GPG Unverified
2B82207D370127A0The key is unknown
1 file · +4 -4
M internal/filtering/service.go +4 -4
| @@ -189,7 +189,7 @@ func (s *ContentBlockingService) injectCosmeticBaseScript(wv *pkgwebkit.WebView) | |||
| 189 | 189 | return | |
| 190 | 190 | } | |
| 191 | 191 | ||
| 192 | - | logging.Info(fmt.Sprintf("[filtering] Injecting cosmetic base script into WebView %d (%d bytes)", wv.ID(), len(baseScript))) | |
| 192 | + | logging.Debug(fmt.Sprintf("[filtering] Injecting cosmetic base script into WebView %d (%d bytes)", wv.ID(), len(baseScript))) | |
| 193 | 193 | ||
| 194 | 194 | // Inject at document-end when DOM is ready for querySelector/MutationObserver | |
| 195 | 195 | userScript := webkit.NewUserScript( | |
| @@ -274,7 +274,7 @@ func (s *ContentBlockingService) applyNetworkFilters(wv *pkgwebkit.WebView, filt | |||
| 274 | 274 | } | |
| 275 | 275 | s.mu.Unlock() | |
| 276 | 276 | ||
| 277 | - | logging.Info(fmt.Sprintf("[filtering] Applied network filters to WebView %d", wv.ID())) | |
| 277 | + | logging.Debug(fmt.Sprintf("[filtering] Applied network filters to WebView %d", wv.ID())) | |
| 278 | 278 | } | |
| 279 | 279 | ||
| 280 | 280 | // setupCosmeticHooks sets up hooks for cosmetic filtering on navigation events. | |
| @@ -318,12 +318,12 @@ func (s *ContentBlockingService) injectCosmeticRules(wv *pkgwebkit.WebView, uri | |||
| 318 | 318 | } | |
| 319 | 319 | ||
| 320 | 320 | // Inject the script | |
| 321 | - | logging.Info(fmt.Sprintf("[filtering] Injecting cosmetic rules for domain: %s (%d bytes)", domain, len(script))) | |
| 321 | + | logging.Debug(fmt.Sprintf("[filtering] Injecting cosmetic rules for domain: %s (%d bytes)", domain, len(script))) | |
| 322 | 322 | pkgwebkit.RunOnMainThread(func() { | |
| 323 | 323 | if err := wv.InjectScript(script); err != nil { | |
| 324 | 324 | logging.Error(fmt.Sprintf("[filtering] Failed to inject cosmetic rules for %s: %v", domain, err)) | |
| 325 | 325 | } else { | |
| 326 | - | logging.Info(fmt.Sprintf("[filtering] Successfully injected cosmetic rules for domain: %s", domain)) | |
| 326 | + | logging.Debug(fmt.Sprintf("[filtering] Successfully injected cosmetic rules for domain: %s", domain)) | |
| 327 | 327 | } | |
| 328 | 328 | }) | |
| 329 | 329 | } | |
| @@ -189,7 +189,7 @@ func (s *ContentBlockingService) injectCosmeticBaseScript(wv *pkgwebkit.WebView) | |||||
| 189 | return | 189 | return | ||
| 190 | } | 190 | } | ||
| 191 | 191 | ||||
| 192 | - | logging.Info(fmt.Sprintf("[filtering] Injecting cosmetic base script into WebView %d (%d bytes)", wv.ID(), len(baseScript))) | 192 | + | logging.Debug(fmt.Sprintf("[filtering] Injecting cosmetic base script into WebView %d (%d bytes)", wv.ID(), len(baseScript))) |
| 193 | 193 | ||||
| 194 | // Inject at document-end when DOM is ready for querySelector/MutationObserver | 194 | // Inject at document-end when DOM is ready for querySelector/MutationObserver | ||
| 195 | userScript := webkit.NewUserScript( | 195 | userScript := webkit.NewUserScript( | ||
| @@ -274,7 +274,7 @@ func (s *ContentBlockingService) applyNetworkFilters(wv *pkgwebkit.WebView, filt | |||||
| 274 | } | 274 | } | ||
| 275 | s.mu.Unlock() | 275 | s.mu.Unlock() | ||
| 276 | 276 | ||||
| 277 | - | logging.Info(fmt.Sprintf("[filtering] Applied network filters to WebView %d", wv.ID())) | 277 | + | logging.Debug(fmt.Sprintf("[filtering] Applied network filters to WebView %d", wv.ID())) |
| 278 | } | 278 | } | ||
| 279 | 279 | ||||
| 280 | // setupCosmeticHooks sets up hooks for cosmetic filtering on navigation events. | 280 | // setupCosmeticHooks sets up hooks for cosmetic filtering on navigation events. | ||
| @@ -318,12 +318,12 @@ func (s *ContentBlockingService) injectCosmeticRules(wv *pkgwebkit.WebView, uri | |||||
| 318 | } | 318 | } | ||
| 319 | 319 | ||||
| 320 | // Inject the script | 320 | // Inject the script | ||
| 321 | - | logging.Info(fmt.Sprintf("[filtering] Injecting cosmetic rules for domain: %s (%d bytes)", domain, len(script))) | 321 | + | logging.Debug(fmt.Sprintf("[filtering] Injecting cosmetic rules for domain: %s (%d bytes)", domain, len(script))) |
| 322 | pkgwebkit.RunOnMainThread(func() { | 322 | pkgwebkit.RunOnMainThread(func() { | ||
| 323 | if err := wv.InjectScript(script); err != nil { | 323 | if err := wv.InjectScript(script); err != nil { | ||
| 324 | logging.Error(fmt.Sprintf("[filtering] Failed to inject cosmetic rules for %s: %v", domain, err)) | 324 | logging.Error(fmt.Sprintf("[filtering] Failed to inject cosmetic rules for %s: %v", domain, err)) | ||
| 325 | } else { | 325 | } else { | ||
| 326 | - | logging.Info(fmt.Sprintf("[filtering] Successfully injected cosmetic rules for domain: %s", domain)) | 326 | + | logging.Debug(fmt.Sprintf("[filtering] Successfully injected cosmetic rules for domain: %s", domain)) |
| 327 | } | 327 | } | ||
| 328 | }) | 328 | }) | ||
| 329 | } | 329 | } | ||