Problem

When going to the mistral provider settings page in Drupal 11.3 you get:

Drupal\ai\Exception\AiResponseErrorException: Something undefined was broken in the response from Mistral AI: Since symfony/http-foundation 7.4: Request::get() is deprecated, use properties ->attributes, query or request directly instead. in Drupal\ai_provider_mistral\Plugin\AiProvider\MistralProvider->errorCatcher() (line 318 of modules/contrib/ai_provider_mistral/src/Plugin/AiProvider/MistralProvider.php).

CommentFileSizeAuthor
#6 mistral-4.diff1.01 KBfago

Issue fork ai_provider_mistral-3564465

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

fago created an issue. See original summary.

fago’s picture

fago
Primary language German
Location Vienna
commented
Title: Drupal 11.3 compatibility: » Drupal 11.3 compatibility: AiResponseErrorException: Request::get() is deprecated
fago’s picture

fago
Primary language German
Location Vienna
commented

Problem is not specific to Request::get(), but that generally various deprecation warnings are already enough to break it. That's a pre-existing issue with errors/warnings/notices in general, however in particular a problem with un-important things like deprecation warnings.

I'd propose to ignore deprecation warnings for now and to take a closer look at a better API-library first before investing more time into this.

fago’s picture

fago
Primary language German
Location Vienna
commented
Version: 1.1.x-dev » 1.0.x-dev

let's fix this in 1.0.x, 1.1.x is unreleased and can be used for new developments like changing the API package.

fago opened merge request !4

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Active » Needs review
StatusFileSize
new mistral-4.diff1.01 KB

submitted MR. attaching a copy of it for usage with composer-patches.

petar_basic’s picture

petar_basic commented
Status: Needs review » Reviewed & tested by the community
fago’s picture

fago
Primary language German
Location Vienna
commented
Assigned: Unassigned » fago

let's check test fails first.

fago’s picture

fago
Primary language German
Location Vienna
commented
Assigned: fago » Unassigned

fails are unrelated. opened #3564675: Make 1.0.x gitlab-ci pipeline green

  • fago committed 0b0a8653 on 1.1.x 
    fix: #3564465 fix AiResponseErrorException raising wrong exceptions due...
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Reviewed & tested by the community » Fixed

Merged!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • fago committed fe0a2145 on 1.0.x 
    fix: #3564465 fix AiResponseErrorException raising wrong exceptions due...

Status: Fixed » Closed (fixed)

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