Skip to content

[8.19] Error if installed plugin is inside plugins folder (#137398)#137509

Merged
mamazzol merged 1 commit intoelastic:8.19from
mamazzol:backport/8.19/pr-137398
Nov 4, 2025
Merged

[8.19] Error if installed plugin is inside plugins folder (#137398)#137509
mamazzol merged 1 commit intoelastic:8.19from
mamazzol:backport/8.19/pr-137398

Conversation

@mamazzol
Copy link
Copy Markdown
Contributor

@mamazzol mamazzol commented Nov 3, 2025

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

* Error if installed plugin is inside plugins folder

If a non directory file is in the plugins folder during installation,
there will be a non descriptive error during findBundle, as it assumes
everything is a directory. This new preemptive check adds a descripive
error message to avoid this scenario.

Also, for consistency and to not use a deprecated URL constructor, in
the download method we now use URI.

Closes elastic#27401

(cherry picked from commit 941f302)
Copy link
Copy Markdown
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mamazzol mamazzol merged commit 26358e3 into elastic:8.19 Nov 4, 2025
23 checks passed
@mamazzol mamazzol deleted the backport/8.19/pr-137398 branch November 4, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants