Forums Bug Reports Thread

API settings surfaces raw app.ini config-key path in tenant-admin copy

Patrick Bass · Jun 6 · 9 · 1 Locked
[Cosmetic] [Low Priority] [Bug Fixed] [Always Reproduces]
🚀 OP Jun 6, 2026 7:51pm

Area: Integration / API (audit p13) · Surface: /admin/api/settings (rate-limit precedence card) · Dimension: Law 6 (config paths on UI) · Severity: cosmetic

Law 6 says users should not see internal mechanism like config file paths. This admin surface exposes 'app.ini' and the raw key 'api.rate_limit_per_minute'. It is a privileged super-admin surface (app.ini is super-admin-writable in this codebase), so the severity is low, but the phrasing leaks implementation detail where 'the platform default' alone would read cleaner and stay on-brand.

Evidence

platform/templates/admin/api-keys/settings.php:63 `<span class="ak-stat-strip__sub">from app.ini</span>`, :97 `…the platform default of <strong><code>…</code></strong> req/min from <code>app.ini</code>.`, :125 `<strong>Platform default</strong> from <code>api.rate_limit_per_minute</code> in <code>app.ini</code> is the floor…`. These name the internal config file and config key directly to the tenant super-admin audience.

Suggested fix. Drop the file/key names from customer-facing copy — say 'the platform default' instead of 'from app.ini' / 'api.rate_limit_per_minute in app.ini'. Keep the numeric value; remove the mechanism.

Filed by the automated tenant-app audit and adversarially evidence-verified. Status: verified. Open — not yet actioned.


Patrick Bass
@mobieus

🚀 Jun 7, 2026 5:44am

Resolved — fixed and deployed. Commit 059d6bf29b6f, shipped dev-first then to all tenants on 2026-06-06.

Removed all three customer-facing references to the config mechanism in /home/patrick/mobieus-io/platform/templates/admin/api-keys/settings.php: stat-strip sub-label 'from app.ini' -> 'platform default'; field hint dropped 'from app.ini'; precedence card dropped 'from api.rate_limit_per_minute in app.ini'. Numeric platform-default value ($iniDefault) is preserved in all three spots.

Status: fixed. Thread closed and locked.


Patrick Bass
@mobieus

Log in or register to reply to this thread.