Conversation
- Introduced a new Varnish service with configuration for caching static assets and handling specific request types. - Updated Rocket.Chat service to depend on Varnish and added necessary Traefik labels for routing. - Created default VCL configuration for Varnish to manage caching behavior.
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #36025 +/- ##
===========================================
- Coverage 64.74% 64.66% -0.08%
===========================================
Files 3156 3306 +150
Lines 104916 108072 +3156
Branches 19960 20315 +355
===========================================
+ Hits 67929 69889 +1960
- Misses 34298 35353 +1055
- Partials 2689 2830 +141
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
…uration - Eliminated Varnish service dependency from Rocket.Chat and removed related configuration in the Docker Compose file. - Adjusted service labels for Traefik routing accordingly.
- Changed Rocket.Chat routing rule to use PathPrefix(`/sockjs`). - Modified Varnish routing rule to exclude paths for `/sockjs` and `/websocket`. - Adjusted Varnish VCL configuration to handle WebSocket connections with `pipe` instead of `pass`.
- Modified Rocket.Chat routing rule to allow health check path alongside `/sockjs`. - Updated Varnish routing rule to exclude health check path from its routing logic.
… Varnish dependency - Added `app-network` to the Docker Compose file for service communication. - Updated Rocket.Chat service to depend on Varnish for improved caching and routing.
1 participant
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments