seo: Environment check for production always returns false if ENVIRONMENT is not set

@ethercreative +1 to merge this PR please! We have had two sites affected by the same.

@Tam, @ethercreative I know you guys are busy, but this seems to cause a lot of issues for several people. I think some people might not even know their site is not indexable anymore, which can cause a lot of damage. Is it possible to take a look at this?

@jacotijssen It would be nice thing to warn potential users on the Craft Plugin store page that this may be abandoned (I think they do that?). It’s got over 16,000 active installs - and really shouldn’t be adding to that number with issues like this one that may catch people unaware.

(I’ve been notified that Craft is reaching out to the developer, and if they don’t hear back in a specified time, they mark it as abandoned.)

I understand this is a free plugin and I hate to say it, but is there a reason why this hasn’t been fixed? The issue seems quite serious and easy to miss until it is too late for those unaware. Easy fixes have been proposed, is there a reason why one cannot be merged?

@cstudios-slovakia we have a site currently on Craft v4.3.3 and the plugin is working by adding ENVIRONMENT alongside CRAFT_ENVIRONMENT in the .env file. Hope this helps as a hotfix for everyone until this is sorted.

CRAFT_ENVIRONMENT=production
ENVIRONMENT=production

+1, please merge this PR 👍

Today I noticed this issue several weeks after a new site launched and countless times of re-validating in Search Console (which unfortunately doesn’t give a reason for not indexing the pages). The only option right now is disabling the plugin alltogether, as the issue is buried in the plugin code. This will also remove the sitemap.xml, which is the reason I use this plugin on this project.

So… the plugin is now officially abandoned. Shame because it was the only free SEO plugin for Craft.

@cstudios-slovakia we have a site currently on Craft v4.3.3 and the plugin is working by adding ENVIRONMENT alongside CRAFT_ENVIRONMENT in the .env file. Hope this helps as a hotfix for everyone until this is sorted.

CRAFT_ENVIRONMENT=production
ENVIRONMENT=production

This works in Craft 4.3.1 with SEO plugin 4.0.3.

+1 to merge PR!!

The plugin should be using Craft::$app->env, rather than worrying about how it was set exactly.

I confirm, the check for the environment is not right in the SEO source - SeoService.

$env = getenv('ENVIRONMENT') ?? getenv('CRAFT_ENVIRONMENT');

==>> getenv(‘ENVIRONMENT’) with a default craft cms .env config (with no ENVIRONMENT config), will always return false (not null), so in the lines bellow will always set the x-robots-tag to ‘none,noimageindex’.

// Always noindex except on production environment
if ($env !== 'production')
{
	$headers->set('x-robots-tag', 'none, noimageindex');
	return;
}

===>> the fix would be reworking the ?? to a check on a false value

So indeed a quick fix is to add ENVIRONMENT=production to your .env, but hope this will be fixed in the future. ENVIRONMENT=production CRAFT_ENVIRONMENT=production

+1

This was supposed to be fixed in 4.1.0, but we’re still getting x-robots-tag: none on production?

Yep, #447.

Does anyone even maintain this anymore?

@ryanmasuga well, with 118 open issues and no updates in the last 9 months, I think we all know the answer to that question 😥

Since this is a open source plugin, I find it very difficult to “report” them, especially because (apart from this issue) I’m very grateful for this plugin. But I also think this might be the only thing left to do.

So I joined you in reporting the plugin and hopefully everyone else here will do the same.

Does anyone even maintain this anymore? This is a big enough issue (that is easy enough to fix) and is not being addressed that I clicked “Report plugin” in the Craft Plugin store (right side: https://plugins.craftcms.com/seo) to see if the Craft devs can figure out if this developer is still around - or warn users it may not be actively maintained.