Account created on 24 March 2015, over 9 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States mikegodin
return [
  KernelEvents::REQUEST => ['doAnonymousRedirect',28],
];
πŸ‡ΊπŸ‡ΈUnited States mikegodin

Created Merge Request #1 to resolve the issue.

πŸ‡ΊπŸ‡ΈUnited States mikegodin

Here's the patch from #7, adjusted for Drupal 10.1.5

πŸ‡ΊπŸ‡ΈUnited States mikegodin

Confirming exactly the same issue as observed by aarantes β†’ when updating from 1.0.0-beta1 to 2.0.0-alpha5, with Drupal: 10.1.5 and PHP: 8.1.16, while running drush updb:

> [notice] Update started: rabbit_hole_update_8103
> [notice] Update completed: rabbit_hole_update_8103
> [notice] Update started: rabbit_hole_update_8104
> [notice] Update completed: rabbit_hole_update_8104
> [notice] Update started: rabbit_hole_update_8105
> [notice] Update completed: rabbit_hole_update_8105
> [notice] Update started: rabbit_hole_update_8106
> [error] 'rh_action' not found
> [error] Update failed: rabbit_hole_update_8106
> [notice] Copied modules/custom/content_type_theme/css/adminimal-custom.css to sites/default/files/adminimal-custom.css.
[error] Update aborted by: rabbit_hole_update_8106
[error] Finished performing updates.

πŸ‡ΊπŸ‡ΈUnited States mikegodin

If it is necessary to install this fork via composer before it gets merged, edit composer.json, replacing:

    "repositories": [
        {
            "type": "composer",
            "url": "https://packages.drupal.org/8"
        }

with

    "repositories": [
        {
            "type": "composer",
            "url": "https://packages.drupal.org/8",
            "exclude": [
                "drupal/openid_connect"
            ]
        },
        {
            "type": "git",
            "url": "https://git.drupalcode.org/issue/openid_connect-3393666.git"
        }

Then require via composer:

composer require 'drupal/openid_connect:dev-3393666-drupal-10-compatibility' --with-all-dependencies

πŸ‡ΊπŸ‡ΈUnited States mikegodin

Noting that the "PHP 7.4 & MySQL 5.7, D10.1.4 Composer require failure" is expected since Drupal 10 requires PHP 8.1.

πŸ‡ΊπŸ‡ΈUnited States mikegodin

The MR 191, when tested locally, passed all PHP Unit tests and all Upgrade Status tests with no deprecation warnings.

πŸ‡ΊπŸ‡ΈUnited States mikegodin

Here's #9 updated for 9.5.x, with the "useless-escape" removed.

Production build 0.69.0 2024