Closed (fixed)
Project:
Guardr
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
24 Oct 2018 at 18:00 UTC
Updated:
14 Sep 2019 at 04:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
damienmckennaComment #3
shrop commentedThis sounds like a good idea as long as we can accomplish two goals:
Seems this may possible work with those to goals, but needs to be tested.
I recommend a branch for this in Guardr and Guardr Core to test the setup.
Any other thoughts/concerns from the community?
Comment #4
joestewart commentedPatch attached. The patch will need to be updated with a release of guardr_core that has the dependencies defined.
Comment #5
joestewart commentedPatch attached. The patch will need to be updated with a release of guardr_core that has the dependencies defined.
Comment #6
joestewart commentedPatch rerolled.
Generated with "composer remove drupal/autologout drupal/diff drupal/hacked drupal/login_history drupal/login_security drupal/mass_pwreset drupal/password_policy drupal/r4032login drupal/seckit drupal/security_review drupal/session_limit drupal/username_enumeration_prevention"
Corresponding patch in guardr_core unaffected - https://www.drupal.org/project/guardr_core/issues/3008975#comment-12839318
Comment #7
damienmckennaSee also: #3059229: Remove the composer.lock file
Comment #8
shrop commentedPlanning to mark this fixed once #3072574: Remove files no longer needed in the Guardr project since the Github project handles D8 one-time installs is reviewed and committed, because the composer.json will be removed now that dependencies are managed in Guardr Core.
Comment #9
shrop commentedNow that #3072574: Remove files no longer needed in the Guardr project since the Github project handles D8 one-time installs is completed we can mark this one fixed. Thanks for all the help everyone!