#2805 closed enhancement (fixed)
[patch] make sitemaps more space effective
| Reported by: | Owned by: | Adrian Holovaty | |
|---|---|---|---|
| Component: | Contrib apps | Version: | 0.95 |
| Severity: | trivial | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The XML generates by sitemaps contains lot of whitespace. Since sitemaps are meant to be only processed by machines and the templates are only seldom edited by humans it makes sense to strip out most unneeded whitespace. Basically this is a readability bandwidth tradeoff. This patch changes the sitemaps framework to use the same approach as the syndication framework: prefer bandwidth over readability. It reduced the size of my sitemap by about 20 %.
Get the Patch at http://c0re.23.nu/c0de/misc/django-3810-sitemap_space_saver.diff
Attachments (1)
Change History (3)
by , 18 years ago
| Attachment: | django-3810-sitemap_space_saver.diff.txt added |
|---|
comment:1 by , 18 years ago
comment:2 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Readability of XML is not important, but readability of templates *is* important. A good solution could be to use the
{% spaceless %}template tag to strip the space on output. I'll check in that change momentarily.Also note that if you're super concerned about bandwidth, just use gzip for your sitemap. See the djangoproject.com URLconf for an example of how to do that.