Apply a custom cache configuration for static assets

[bpetetot]

I wanted to be more aggressive on my font cache by explicitly adding this configuration:

createExpressApp({
  configure: (app) => {
    // ...
    // Cache assets
    app.use('/fonts', express.static('build/client/fonts', { immutable: true, maxAge: '1y' }));
  }
})

But the server's default configuration for static files (for build/client) is made with a maxAge: '1h'.

remix-express-vite-plugin/packages/remix-create-express-app/src/index.ts

Lines 90 to 94 in 03028a9

	app.use(
	express.static(isProductionMode ? `${buildDirectory}/client` : 'public', {
	maxAge: '1h',
	}),
	)

So we can't replace cache-control for specific static files (like fonts) with a custom configuration.

Could we have the default configuration for static files after the custom configuration is applied? (I can do the PR if needed)

Thank you for this awesome project 🔥

[kiliman]

Sure, I should probably allow you to provide defaults.

[kiliman]

If you have time to provide a PR, that would be great; otherwise, I probably won't get to it until this weekend. You can always use patch-package if it's a blocker for you.

[bpetetot]

Thanks kiliman, it can be done later, it's not a blocker for me.

[kiliman]

Although not exactly the cache setting we discussed, v0.4.5, which was just released, should fix this. Your configure function will be called before I set the static paths, so your specific /fonts entry will be handled before the default.

[bpetetot]

Great! Thanks 👍