GuzzleHTTP constraint doesn't allow upgrade from Drupal 9 to 10.

Created on 2 November 2023, about 1 year ago

Problem/Motivation

Trying to upgrade to Drupal 10 and the upstream dependency on dakkusingh/oktasdk-php is not allowing the GuzzleHTTP:^7

Proposed resolution

Change the composer dependency to a supported fork here: https://github.com/pfrilling/oktasdk-php

πŸ› Bug report
Status

Active

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States pfrilling Minster, OH

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @pfrilling
  • @pfrilling opened merge request.
  • Status changed to Fixed about 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States pfrilling Minster, OH

    This has been fixed and merged into the 8.x-2.x-dev branch

  • πŸ‡ΉπŸ‡³Tunisia medwassim

    Hello,
    i'm still facing this issue :

    /var/www/html $ composer require 'drupal/okta_api:2.x-dev@dev' -W
    ./composer.json has been updated
    Running composer update drupal/okta_api --with-all-dependencies
    Gathering patches for root package.
    > DrupalProject\composer\ScriptHandler::checkComposerVersion
    Loading composer repositories with package information
    Updating dependencies
    Your requirements could not be resolved to an installable set of packages.

    Problem 1
    - dakkusingh/oktasdk dev-master requires guzzlehttp/guzzle ^6.1 -> found guzzlehttp/guzzle[6.1.0, ..., 6.5.x-dev] but it conflicts with your root composer.json require ( ^7.5).
    - drupal/okta_api 2.x-dev is an alias of drupal/okta_api dev-2.x and thus requires it to be installed too.
    - drupal/okta_api dev-2.x requires dakkusingh/oktasdk dev-master -> satisfiable by dakkusingh/oktasdk[dev-master].
    - Root composer.json requires drupal/okta_api 2.x-dev@dev -> satisfiable by drupal/okta_api[2.x-dev (alias of dev-2.x)].

    Installation failed, reverting ./composer.json and ./composer.lock to their original content.

    Thank you

  • πŸ‡ΉπŸ‡³Tunisia medwassim

    hello @pfrilling
    Any help for above comment ?
    the dakkusingh/oktasdk dev-master still have dependency to guzzle 6.
    Thank you

  • πŸ‡ΊπŸ‡¦Ukraine Oleksiy Lutsk πŸ‡ΊπŸ‡¦

    Hello,

    The solution is to add:

      "repositories": [
        {
          "type": "vcs",
          "url": "https://github.com/pfrilling/oktasdk-php.git"
        }
      ],

    to your project composer.json file to let it find the correct version of dakkusingh/oktasdk

  • πŸ‡ΉπŸ‡³Tunisia medwassim

    Thanks @Oleksiy
    the patch "Add a group option to the user creation #3": "https://patch-diff.githubusercontent.com/raw/dakkusingh/oktasdk-php/pull..." is rejected.
    Thanks

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

  • Status changed to Fixed 10 months ago
  • πŸ‡­πŸ‡ΊHungary SunnyGambino Buj

    Hi All,
    #6 is the solution. May I suggest to add it somewhere in the documentation please?

  • πŸ‡ΊπŸ‡ΈUnited States jldust

    I'm also running into the issue called out in #7 when adding it to the composer file. That appears to also have an open issue here: https://www.drupal.org/project/okta_api/issues/3418721 πŸ“Œ Oktasdk-php patch is no longer needed Active

  • πŸ‡¨πŸ‡¦Canada bonfil1 British Columbia

    I'm facing same issue as #10 and #7, i can't apply the patch in #3418721 as per patch level this is apply after the patch we're trying to avoid, is there any way to merge #3418721?

Production build 0.71.5 2024