fix: remove CRAFT_WEB_URL, fixes #5665

[timkelty]

The Issue

The CRAFT_WEB_URL env variable is not necessary for a Craft project, and prevents multi-site features from working.

How This PR Solves The Issue

Does not manage CRAFT_WEB_URL.

It also removes the comment about backward compatibility, as it isn't quite true. While PRIMARY_SITE_URL isn't an "official" var that Craft uses, it is a de-facto standard, as Craft's installer script will attempt to set it, and it is helpful to have the primary site set to reference in your config.

Manual Testing Instructions

Start a craftcms ddev project:

ddev config --project-type=craftcms --docroot=web --create-docroot
touch .env
ddev start

Ensure CRAFT_WEB_URL was not written to .env (command should have no output):

cat .env | grep "CRAFT_WEB_URL="

Automated Testing Overview

Related Issue Link(s)

Fixes #5665

Release/Deployment Notes

[github-actions]

Download the artifacts for this pull request:

• all-ddev-executables.zip
• ddev-linux-amd64.zip
• ddev-linux-arm64.zip
• ddev-macos-amd64.zip
• ddev-macos-arm64.zip
• ddev-windows-amd64-installer.zip

See Testing a PR

[rfay]

If CRAFT_WEB_URL isn't necessary for an ordinary project then I'm all in favor of not having it added to .env.

[rfay]

Please add manual testing instructions (and do them) thanks!

[timkelty]

@rfay added manual testing instructions

[stasadev]

This looks fine to me.

And I see that CraftCMS uses PRIMARY_SITE_URL since 2020 craftcms/cms@7e18bc5