There is an extra "on" in the comment for the variable $cookie_domain in default.settings.php. Pretty sure it was not intended.
The attached patch removes it.

CommentFileSizeAuthor
#4 1040190-settings-comment-cookie_domain-typo.patch1.29 KBTor Arne Thune
settings-comment-cookie_domain-typo.patch946 bytesTor Arne Thune
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Tor Arne Thune’s picture

Tor Arne Thune CreditAttribution: Tor Arne Thune commented
Component: install system » documentation
Issue tags: +Quick fix
jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community

Good catch, good patch, thanks!

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Reviewed & tested by the community » Needs work

the wrapping needs to be updated.

Tor Arne Thune’s picture

Tor Arne Thune CreditAttribution: Tor Arne Thune commented
Status: Needs work » Needs review
FileSize
1040190-settings-comment-cookie_domain-typo.patch1.29 KB

Good eye. The attached patch should have correct wrapping.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Great, thanks!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

+1

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Darn. Too bad we didn't catch this before 7.0. Various things freak out when they need to update settings.php in a minor release.

Nonetheless, good catch. Committed to HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Quick fix

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.