- 🇺🇸United States shrop
Appreciate all the work in this issue. I think we need to work on some changes and this covers many of them. What do you think about working on this for a 3.x version of Guardr Core? Seems like that would be a good break. Right now, I would like to get Guardr 2.x to D10 compatibility and then we could do this as a D10 compatible version at 3.x.
- 🇬🇧United Kingdom the_g_bomb
This MR returns many of the modules from the D7 version. At a guess, they were removed as D8 versions weren't available at the time. I haven't done any work to split out the modules yet as I thought a conversation was probably needed about how to split them out and what approach to take. Do we remove the modules for the Guardr audit and make a new module? Or do we make a submodule and get all the dependencies in at once?
As it stands, this MR is basically a D9 version of the module as it was in D7.
- @the_g_bomb opened merge request.
- 🇬🇧United Kingdom the_g_bomb
This can be tested using:
composer create-project --no-interaction --no-install drupal/recommended-project:^9.5 guardrtest cd guardrtest composer config repositories.guardr_core vcs git@git.drupal.org:issue/guardr_core-3159974.git composer require drupal/guardr_core:dev-3159974-split-guardr-into -W - 🇬🇧United Kingdom the_g_bomb
Closing the PR, until a direction can be determined.
I would like to see the some of the modules that were included in D7 guardr back in the module, at least. Most are now available for D8/9 when the 8.x branch was created, I suspect.
This list is now currently out of date.
https://www.drupal.org/docs/distributions/guardr/security-features-and-s... →