ampproject / ampproject/amp-wp

LightSpeed Cache comment should be omitted in special case

Open
#7,967 0 comments 1 reaction 0 assignees View on GitHub
Enhancement
Dominant language
PHP
Stars
1.8k
Forks
378
Avg merge
1d 15h
Merged PRs (30d)
10

Description

As noted by @milindmore22 in https://github.com/litespeedtech/lscache_wp/issues/212#issuecomment-2569443269, a lot of support topics still involve the LightSpeed Cache comment. While waiting for the issue to be fixed upstream, we should be able to resolve it by adding the following (h/t @szepeviktor in https://github.com/litespeedtech/lscache_wp/issues/212#issuecomment-2569524293):

```php
add_filter( 'litespeed_comment', '__return_false' );
```

Inside this condition:

https://github.com/ampproject/amp-wp/blob/76ab0ae10712638b23568af9bb0ed12c1465600d/includes/validation/class-amp-validation-manager.php#L632-L640

Contributor guide

Open the contributing guide

Research direction

Start at includes/validation/class-amp-validation-manager.php around lines 632-640 and inspect the condition described in the issue. Confirm the LightSpeed-specific filter is applied only in that special case, and verify that the LightSpeed Cache comment is omitted there without changing other validation behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.