Skip to content

Fix: resolve login page assets for marketplace installs (GLPI 12) - #246

Merged
Rom1-B merged 1 commit into
mainfrom
support_47188_12
Oct 27, 2026
Merged

Rom1-B merged 1 commit into
mainfrom
support_47188_12

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !47188
  • When the plugin is installed in a marketplace directory located outside the GLPI root (e.g. GLPI_MARKETPLACE_DIR set to /var/lib/glpi/marketplace), the login page requested its CSS and JS with the server filesystem path as URL.
  • The files returned a 404, the alerts were unstyled and the server path was disclosed in the browser console.
  • Assets are now loaded through the /plugins/news/ route, which works wherever the plugin is installed.
  • Also reorders the CHANGELOG: entries left in a stale Unreleased section are moved to 1.15.0, where they were released.
Refused to apply style from 'https://glpi.example.com/var/lib/glpi/marketplace/news/css/styles.css?v=...' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.
GET https://glpi.example.com/var/lib/glpi/marketplace/news/js/news.js?v=... net::ERR_ABORTED 404 (Not Found)

Screenshots (if appropriate):

@Rom1-B
Rom1-B requested review from a team, Lainow, RomainLvr and jdurand-teclib and removed request for a team October 27, 2026 08:30
@Rom1-B
Rom1-B merged commit 1617f53 into main Oct 27, 2026
7 checks passed
@Rom1-B
Rom1-B deleted the support_47188_12 branch October 27, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants