File usage tracking authentication issue on multilingual sites

Open on Drupal.org →

Created on 13 January 2025, 18 days ago

Problem/Motivation

The "file tracking usage" url as displayed on the /admin/config/media/kontainer configuration page does not seem to work on multilingual sites. For example: https://example.com/en/kontainer/api/file-usages.

When performing a GET request with bearer token via postman:

https://example.com/kontainer/api/file-usages: 200 OK
https://example.com/en/kontainer/api/file-usages: 403 Forbidden

The kontainer_auth authentication provider is only applied to the file tracking endpoint when it exactly matches the /kontainer/api/file-usages route definition. When it's not applied, the kontainer_auth_role is never assigned, and every request will return a 403.

Proposed resolution

Tweak the KontainerAuth::applies() function to support multilingual paths.

🐛 Bug report

Status

Active

Version

1.1

Component

Code

Created by

🇧🇪Belgium robindh

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!12File usage tracking authentication issue on multilingual sites
Merged
🇸🇮Slovenia bcizej
updated 9 days ago
!10File usage tracking authentication issue on multilingual sites
Closed
🇧🇪Belgium robindh
updated 11 days ago

Comments & Activities

Issue created by @robindh
Merge request !10Refactor auth provider applies check to make it support multilingual sites → (Closed) created by robindh
Comment 18 days ago →
🇧🇪Belgium robindh
Note: the str_ends_with call used in the merge request is only supported on php ^8.0
First commit to issue fork.
Merge request !12Resolve #3499401 "File usage tracking" → (Merged) created by bcizej
Comment 11 days ago →
System Message
bcizej → closed merge request !10
Comment 11 days ago →
🇸🇮Slovenia bcizej
Thanks for the PR. I've cherry picked your commit to the feature branch (the commit was on the 1.x branch) and used a PHP 7 compatible function as the module still has Drupal 9 compatibility listed and that version supports PHP 7.
Comment 9 days ago →
🇧🇪Belgium robindh
Alright, that seems fair; I've tested the tweaked PHP7 compatibility patch - works as expected. Marking as RTBC

Comment 9 days ago →

bcizej → committed f4a5609e on 1.x

Issue #3499401 by robindh, bcizej: File usage tracking authentication...

Comment 9 days ago →
🇸🇮Slovenia bcizej
Merged, thanks.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024