Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add no-cache flag #727

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add no-cache flag #727

wants to merge 1 commit into from

Conversation

querielo
Copy link

The PR allows us to disable browser cache. It can be suitable if we often change served content.

Copy link

@davidmroth davidmroth left a comment

Choose a reason for hiding this comment

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

Changes look good!

@@ -63,6 +63,8 @@ export const startServer = async (

if (args['--cors'])
response.setHeader('Access-Control-Allow-Origin', '*');
if (args['--no-cache'])
Copy link
Member

Choose a reason for hiding this comment

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

Could we add a test please?

@LinusU
Copy link

LinusU commented Jul 24, 2023

This would be much appreciated 🙏

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants