Skip to content

Conversation

@lcomplete
Copy link
Owner

Summary

  • Implemented custom WebResourceCacheConfig for better control over static resource caching
  • Changed ReactAppController to use forward instead of direct path return
  • Removed default Spring caching configuration from application.yml

Test plan

  • Static resources are properly cached
  • React routing works correctly with forwarding
  • Cache headers are properly set

🤖 Generated with Claude Code

lcomplete and others added 2 commits January 11, 2026 03:03
Refactored web resource caching to use a custom WebResourceCacheConfig
for better control over cache headers. Changed ReactAppController to
use forward instead of direct path to ensure proper cache handling.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@augmentcode
Copy link

augmentcode bot commented Jan 10, 2026

🤖 Augment PR Summary

Summary: This PR refines how the Spring Boot backend serves and caches the embedded React UI’s static resources.

Changes:

  • Adds WebResourceCacheConfig with explicit Cache-Control policies for /index.html, JS/CSS bundles, media assets, and common root-level icons.
  • Updates ReactAppController to forward:/index.html so SPA routes resolve via the static resource pipeline.
  • Removes the previous global spring.web.resources.cache.cachecontrol.max-age setting from application.yml in favor of the custom handler rules.
  • Adjusts mobile layout/navigation CSS to use viewport-height containers, safe-area-aware bottom padding, and an explicit scroll container.

Technical Notes: Caching is now pattern-specific (rather than a single global max-age), enabling different cache behavior for the app shell vs. versioned assets.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@lcomplete lcomplete merged commit d445196 into main Jan 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants