Field validation

Open on Drupal.org →

Created on 2 December 2024, 3 months ago

Problem/Motivation

As user I can provided "feed://test.com/feed-test" as remote URL what generates a fatal error

Module uses https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Component%21Util... which allows to type feed uri.

Steps to reproduce

Edit entity where file_url exists
Type "feed://test.com/feed-test" as remote URL,
press save
visit the entity, system generates a fatal error

InvalidArgumentException: The URI 'feed://test.com/feed-test' is invalid. You must use a valid URI scheme. Use base: for a path, e.g., to a Drupal file that needs the base path. Do not use this for internal paths controlled by Drupal. in Drupal\Core\Utility\UnroutedUrlAssembler->assemble() (line 65 of core/lib/Drupal/Core/Utility/UnroutedUrlAssembler.php).

Proposed resolution

Update validation, allow to use only ftp, http, https protocols

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇵🇱Poland alorenc Wolsztyn, 🇵🇱

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @alorenc
Comment 3 months ago →
🇮🇳India arunsahijpal
Hi @alorenc,
When i am making a field of file_url type, i am encountering this error and it crashes the site
TypeError: Drupal\file_url\Plugin\Field\FieldType\FileUrlFieldItemList::defaultValuesForm(): Return value must be of type array, none returned in Drupal\file_url\Plugin\Field\FieldType\FileUrlFieldItemList->defaultValuesForm() (line 47 of /app/web/modules/contrib/file_url/src/Plugin/Field/FieldType/FileUrlFieldItemList.php).
Are you talking about this?
Comment 3 months ago →
🇵🇱Poland alorenc Wolsztyn, 🇵🇱
Your error is handled by https://www.drupal.org/project/file_url/issues/3490692 🐛 PHP error FileUrlGenerator Active , see https://git.drupalcode.org/project/file_url/-/merge_requests/8#note_419768

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024