Migrating multiline env config due to abra limitations #30

New Issue

2022-01-18T14:46:30Z

decentral1se commented

2022-01-18 14:46:30 +00:00

Unfortunately, due to upstream issues in a lib that abra uses, the multiline env var stuff is broken. it would be handy if we could migrate this config and then just point to this limitation. then wordpress recipe maintainers could keep using the latest abra.

		wordpress/.env.sample
		Lines 14 to 30 in 57122cd677
	
				# Multisite

				#WORDPRESS_CONFIG_EXTRA="\

				#	define('WP_CACHE', false);\

				#	define('WP_ALLOW_MULTISITE', true );"

				# Multisite phase 2 (see README)

				#WORDPRESS_CONFIG_EXTRA="\

				#	define('WP_CACHE', false);\

				#	define('WP_ALLOW_MULTISITE', true );\

				#	define('MULTISITE', true);\

				#	define('SUBDOMAIN_INSTALL', true);\

				#	define('DOMAIN_CURRENT_SITE', '${DOMAIN}');\

				#	define('PATH_CURRENT_SITE', '/');\

				#	define('SITE_ID_CURRENT_SITE', 1);\

				#	define('BLOG_ID_CURRENT_SITE', 1);\

				#	define('FORCE_SSL_ADMIN', true );\

				#	define('COOKIE_DOMAIN', \$_SERVER['HTTP_HOST']);"

needs to be changed.

I have an idea and will test on a test environment and report back with a PR if we're good.

Unfortunately, due to upstream issues in a lib that abra uses, the multiline env var stuff is broken. it would be handy if we could migrate this config and then just point to this limitation. then wordpress recipe maintainers could keep using the latest abra. https://git.coopcloud.tech/coop-cloud/wordpress/src/commit/57122cd677a9066b719f21761d422992314dd5ec/.env.sample#L14-L30 needs to be changed. I have an idea and will test on a test environment and report back with a PR if we're good.

decentral1se referenced this issue from toolshed/organising

2022-01-20 15:46:32 +00:00

document multiline env limitation ("can't separate key from value ") #291

decentral1se commented

2022-03-08 14:08:37 +00:00