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

Support sitemap Etag Header #80

Closed
sonofmagic opened this issue Sep 2, 2019 · 2 comments
Closed

Support sitemap Etag Header #80

sonofmagic opened this issue Sep 2, 2019 · 2 comments

Comments

@sonofmagic
Copy link

What problem does this feature solve?

Sitemap with Rtag can speed up search engine crawling. Baidu search engine can respond to sitemap updates more quickly. refer:https://ziyuan.baidu.com/college/articleinfo?id=456

What does the proposed changes look like?

options:{
Etag:true
}

This feature request is available on Nuxt community (#c63)
@ghost ghost added the cmty:feature-request label Sep 2, 2019
@NicoPennec
Copy link
Member

I was not aware of this optimization by Etag Header optmization for the Baidu search engine 👍

We should be able to take inspiration from this code from Nuxt: https://github.com/nuxt/nuxt.js/blob/76b1167d27b39a19ae2556334cd59e04ae19c1a8/packages/server/src/middleware/nuxt.js#L39-L49

@NicoPennec NicoPennec added this to the 2.1 milestone Sep 26, 2019
@NicoPennec NicoPennec modified the milestones: 2.x, 2.0 Apr 5, 2020
@NicoPennec
Copy link
Member

The ETag is now enabled by default, since release v2.2.0.
To override the default config, see docs

Thank @sonofmagic for your proposal 🙏
It's a useful enhance for performance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants