Drupal 9 compatibility [#3174970]

Comment	File	Size	Author
#2	3174970-2-d9-compat.patch	1.59 KB	drupalninja99

Comment #1

5 October 2020 at 16:07

drupalninja99 created an issue. See original summary.

Log in or register to post comments

Comment #2

drupalninja99 commented 5 October 2020 at 16:12

Status:

Active

» Needs review

Status	File	Size
new	3174970-2-d9-compat.patch	1.59 KB

This patch includes a couple extra things so that the composer.json can be tested independently of the "parent" Guardr project composer. I pulled in a couple of recommendations from https://www.drupal.org/project/rain/issues/3092793#comment-13788700 to see if we can get this working.

I was able to generate a lock file with these updates.

Log in or register to post comments

Comment #3

drupalninja99 commented 5 October 2020 at 16:14

Applied patch to 9.x-test branch in this project.

Log in or register to post comments

Comment #4

joestewart commented 9 October 2020 at 19:49

Updated the 9.x-test that was able to be installed with composer and Drupal 9.0.7.

Updates

* login_security 2.0.0
* composer_security_checker 1.x-dev

Removed

* x_originating_ip
* r4032login
* mass_pwreset
* login_history
* hacked
* security_review 2912708 patch no longer required

Log in or register to post comments

Comment #5

joestewart commented 20 October 2020 at 16:00

Added https://www.drupal.org/project/x_originating_ip/releases/8.x-1.2 to 9.x-test branch.
https://git.drupalcode.org/project/guardr_core/-/tree/9.x-test

Log in or register to post comments

Comment #6

joestewart commented 20 November 2020 at 16:20

Added r4032login back as a composer dependency as the 2.0 release is compatible.
https://www.drupal.org/project/r4032login/releases/2.0.0

Log in or register to post comments

Comment #7

joestewart commented 20 November 2020 at 17:25

Other module Drupal 9 status

Mass Password Reset
https://www.drupal.org/project/mass_pwreset/issues/3097578 - RTBC

Login History
https://www.drupal.org/project/login_history/issues/3042861#comment-1391... - Needs Review

Hacked
No patch for compatibility but two patches address deprecations
https://www.drupal.org/project/hacked/issues/3075667
https://www.drupal.org/project/hacked/issues/3147733

Log in or register to post comments

Comment #8

joestewart commented 20 November 2020 at 21:01

Hacked module issue
https://www.drupal.org/project/hacked/issues/318378 - Needs Review

Log in or register to post comments

Comment #9

joestewart commented 4 December 2020 at 17:51

Status Update:

Add Drupal 9 compatible Login History 8.x-1.1-rc1 to 9.x-test branch

Hacked
https://www.drupal.org/project/hacked/issues/3075667#comment-13927079 Needs Review

Mass Password Reset
https://www.drupal.org/project/mass_pwreset/issues/3097578 - RTBC

Log in or register to post comments

Comment #10

joestewart commented 25 January 2021 at 15:35

Removed composer_security_checker.
#3194045: Remove composer_security_checker

Log in or register to post comments

Comment #11

joestewart commented 28 January 2021 at 20:23

Changed username_enumeration_prevention to dev version due to #3167106: Deprecated function User::getUsername

Log in or register to post comments

Comment #12

joestewart commented 7 May 2021 at 16:17

Changed username_enumeration_prevention to stable release since #3167106: Deprecated function User::getUsername is now in release 1.2.

Status update:

Once drupal/mass_pwreset
https://www.drupal.org/project/mass_pwreset/issues/3097578 is merged and released we can release we can add it back for Drupal 9.

This would allow us to create a merge request and release the same version for Drupal 8 and 9.
Two packages would be removed - hacked and drupal/composer_security_checker.