A customer asked me to compress the sitemap so I implement it in this module. I hope this will help someone.

CommentFileSizeAuthor
#4 compress_sitemap_using-2882626-4.patch3.2 KBDiDebru
#2 compress_sitemap_using-2882626-2.patch3.11 KBDiDebru
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Insasse created an issue. See original summary.

DiDebru’s picture

DiDebru
Primary language German
CreditAttribution: DiDebru at UEBERBIT GmbH commented
FileSize
compress_sitemap_using-2882626-2.patch3.11 KB

Status: Needs review » Needs work

The last submitted patch, 2: compress_sitemap_using-2882626-2.patch, failed testing. View results

DiDebru’s picture

DiDebru
Primary language German
CreditAttribution: DiDebru at UEBERBIT GmbH commented
Status: Needs work » Needs review
FileSize
compress_sitemap_using-2882626-4.patch3.2 KB

Patch rerolled at current 8.x-2.x

sagesolutions’s picture

sagesolutions CreditAttribution: sagesolutions commented

I think a better way compress the sitemap file would be configuring the web server (nginx / apache / whatever) to compress .xml files instead of compressing it in the module.

Thoughts?

gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented
Status: Needs review » Active

I am not sure about the need of sitemap compression. Chunking the sitemap (which is an option in this module) should be a viable measure to reduce load. How beneficial is it to hold a compressed text file in the database anyway?

gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented

[erroneous duplicate comment]

gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented
Status: Active » Postponed (maintainer needs more info)

@sagesolutions This module does not produce files. It holds the XML data inside of the DB, so if we would implement this, we would have to implement the compression logic in the module.

Waiting for feedback on comment #6.

DiDebru’s picture

DiDebru
Primary language German
CreditAttribution: DiDebru at UEBERBIT GmbH commented

Actually I thought it would be a good way to compress the file, I never come in touch with this so I don't know how to do it, so I thought I would use this module for it. Maybe there is a better way to do it.

gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented

Thanks @Insasse, going to leave this as 'postponed', need more input from the community on this.

gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented
Version: 8.x-2.x-dev » 8.x-3.x-dev
gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and at gbyte commented
gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and at gbyte commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)

Closing as per lack of interest.