Add link to dblog if enabled

Created on 27 January 2025, 3 months ago

Problem/Motivation

In 📌 Reduce amount of information relayed on error to the front-end Active , we changed the message that we show to users if there is an error. Part of that message is: See the error log for details.

It'd be great if we could detect if the dblog module is enable and offer a link in that message.

Steps to reproduce

You can do the changes seen in this commit https://git.drupalcode.org/project/project_browser/-/merge_requests/685/... to force the error (just don't commit them).

Proposed resolution

Detect if dblog is enabled, and in that case have the words error log point at it, otherwise leave the message as is.

Feature request
Status

Active

Version

2.0

Component

Code

Created by

🇪🇸Spain fjgarlin

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @fjgarlin
  • 🇮🇳India sayan_k_dutta

    Working on it.

  • Pipeline finished with Failed
    3 months ago
    Total: 557s
    #413219
  • Pipeline finished with Success
    3 months ago
    Total: 419s
    #413222
  • 🇮🇳India sayan_k_dutta

    Made the required changes. Please review MR !704.

  • Pipeline finished with Failed
    3 months ago
    Total: 473s
    #419590
  • Pipeline finished with Failed
    3 months ago
    Total: 455s
    #419595
  • Pipeline finished with Success
    3 months ago
    Total: 528s
    #419602
  • 🇮🇳India sayan_k_dutta

    Made the changes requested. Please review.

  • 🇺🇸United States phenaproxima Massachusetts

    Gave this a quick manual testing using the error-forcing method described in the issue summary.

    Without dblog enabled, I see this when fetching modules:

    And with it:

    That's pretty straightforwardly what I'd expect. The "error log" link takes me to the error log as promised, and the first thing in it is the HTTP error that came back from d.o. Definitely useful for forensic purposes.

    I'd say this is pretty clearly good to go.

  • 🇺🇸United States phenaproxima Massachusetts
  • First commit to issue fork.
  • Pipeline finished with Success
    3 months ago
    Total: 772s
    #421054
  • Pipeline finished with Skipped
    3 months ago
    #421068
  • 🇺🇸United States chrisfromredfin Portland, Maine

    nice little ux improvement here!

  • Automatically closed - issue fixed for 2 weeks with no activity.

  • Pipeline finished with Failed
    2 months ago
    Total: 411s
    #433782
  • Pipeline finished with Failed
    2 months ago
    Total: 372s
    #433788
  • Pipeline finished with Failed
    2 months ago
    Total: 1526s
    #433791
  • Pipeline finished with Failed
    2 months ago
    Total: 1263s
    #433824
  • Pipeline finished with Success
    2 months ago
    Total: 1376s
    #434032
  • Pipeline finished with Skipped
    2 months ago
    #434061
Production build 0.71.5 2024