#2371587: Review install.txt removed some important information from the .txt files that ship with Drupal core, particularly information about how to subscribe to or report security issues.

Let's add it back. I think there's some logic to its original location in INSTALL.txt, but based on feedback in that issue README.txt is probably a better place.

CommentFileSizeAuthor
#2 readme-2426901-2.patch1.15 KBDavid_Rothstein
#1 readme-2426901-1.patch1.13 KBDavid_Rothstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Status: Active » Needs review
FileSize
readme-2426901-1.patch1.13 KB

Here's a patch that simply adds it to README.txt as it originally was in INSTALL.txt.

This could probably be improved, but let's make sure to get something committed here. Information about how to subscribe to or report security issues is important to have in the codebase.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
FileSize
readme-2426901-2.patch1.15 KB

Let's at least switch the links from http://drupal.org to https://www.drupal.org though, in line with changes that were made elsewhere.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Needs review » Reviewed & tested by the community

@David_Rothstein thanks for catching this. Looks good.

jhodgdon’s picture

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

Great! +1. Actually I'll just commit this since @alexpott RTBCd.

  • jhodgdon committed 56ad560 on 8.0.x 
    Issue #2426901 by David_Rothstein: Add back various links (especially...

Status: Fixed » Closed (fixed)

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