fix: Use production url instead of environment variable to enable asset logging [WPB-22566] (#19971) #20008
Conversation
…et logging [WPB-22566] (#19971) * fix: Use production url instead of environment variable to enable asset logging [WPB-22566] * use backend rest * change naming * move isAuditLogEnabledForBackend
|
There was a problem hiding this comment.
Pull request overview
This PR is a cherry-pick of #19971 that fixes asset audit logging configuration. The change replaces the use of environment variable (getWebEnvironment().isProduction) with a direct backend URL check to control whether asset logging is enabled. This ensures asset logging is disabled specifically on the wire.com cloud deployment while remaining enabled on other deployments (staging, internal, self-hosted), regardless of the webapp build type.
Key Changes:
- Replaced environment-based check with backend URL comparison for controlling asset audit logging
- Removed dependency on
getWebEnvironment().isProductionfor audit log decisions
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/script/repositories/team/TeamState.ts | Added static method isAuditLogEnabledForBackend() that checks if the backend URL is not the production cloud backend |
| src/script/repositories/conversation/MessageRepository.ts | Updated asset logging logic to use new backend check and removed unused getWebEnvironment import |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #20008 +/- ##
==========================================
- Coverage 43.69% 43.41% -0.28%
==========================================
Files 1309 1298 -11
Lines 32861 32634 -227
Branches 7286 7247 -39
==========================================
- Hits 14359 14169 -190
+ Misses 16778 16750 -28
+ Partials 1724 1715 -9 🚀 New features to boost your workflow:
|
Pull Request
Summary
Cherry pick of fix: Use production url instead of environment variable to enable asset logging [WPB-22566] #19971
none.
Security Checklist (required)
Accessibility (required)
Standards Acknowledgement (required)