Migrating multiline env config due to abra limitations #30

Closed
opened 2022-01-18 14:46:30 +00:00 by decentral1se · 1 comment
Owner

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.

.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.
Author
Owner
https://git.coopcloud.tech/coop-cloud/wordpress/commit/08c56a2ad94b1ec724e0fb4cf6890c6e659b1772
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/wordpress#30
No description provided.