Rather than having a custom template for the debugging data, I think it would be good to output headers for the debugging data, much like using http.response.debug_cacheability_headers: true.

Comments

Dave Reid created an issue. See original summary.

dave reid’s picture

Category: Task » Feature request
Status: Active » Needs review
StatusFileSize
new5.17 KB
dave reid’s picture

dave reid’s picture

Should this rely on the http.response.debug_cacheability_headers: true parameter instead of this setting?

dave reid’s picture

https://www.drupal.org/project/xmlsitemap/issues/2810833#comment-12834311 is another example of why I think this is good to do, a user had the developer settings enabled and thought the module was broken because they were seeing the debugging output instead of the headers.

dave reid’s picture

StatusFileSize
new5.17 KB
dave reid’s picture

StatusFileSize
new8.84 KB

Fixed a couple values.

  • Dave Reid committed 79cfd85 on 8.x-1.x
    Issue #3007089 by Dave Reid: Use headers to output debugging data,
    
dave reid’s picture

Version: 8.x-1.x-dev » 7.x-2.x-dev
Status: Needs review » Patch (to be ported)

Committed #7 to 8.x-1.x. Moving to 7.x-2.x for consideration.