Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

<!-- served by Surge: timestamp --> #27

Open
MadtownLems opened this issue Mar 2, 2023 · 2 comments
Open

<!-- served by Surge: timestamp --> #27

MadtownLems opened this issue Mar 2, 2023 · 2 comments

Comments

@MadtownLems
Copy link

MadtownLems commented Mar 2, 2023

I believe that most caching plugins of this nature add an HTML comment such as:
<!-- served by Surge: {timestamp of when page was cached} -->

to the end of each page. I was hoping you'd consider adding it here. It would make it easier to verify the cache is working as expected, as well as help debug issues or answer questions about when a page was cached.

Great plugin! It's the first of its kind that I'm giving serious consideration to using in a Production environment.

@kovshenin
Copy link
Owner

Thanks for the suggestion @MadtownLems!

Not a huge fan of altering the resulting output. I think I'd rather put more work into the Site Health dashboard for configuration debugging. However, I will certainly consider this for any future updates. Thank you for trying out Surge!

@iandunn
Copy link

iandunn commented Mar 20, 2023

One alternative you can use is the X-Cache header that Surge outputs. It'll have a value of hit, miss, or expired. IMO that's just as easy to see as viewing the source, and I agree w/ Konstantin that it's nice to not alter the output.

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

No branches or pull requests

3 participants