Implement hook_requirements()

Created on 12 April 2023, over 1 year ago
Updated 25 July 2023, over 1 year ago

Problem/Motivation

Certificate expiration date is not displayed on /admin/reports/status page.

Steps to reproduce

Proposed resolution

Implement hook_requirements() for reporting configuration issues or certificate expiration date on /admin/reports/status page.

Remaining tasks

User interface changes

API changes

Data model changes

Feature request
Status

Fixed

Version

5.0

Component

Code

Created by

🇭🇺Hungary mxr576 Hungary

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

Comments & Activities

  • Issue created by @mxr576
  • Assigned to akshaydalvi212
  • 🇮🇳India akshaydalvi212

    i will work on implementing the hook_requirements().

  • @akshaydalvi212 opened merge request.
  • Issue was unassigned.
  • 🇮🇳India akshaydalvi212

    I m afraid to say this and I'm sorry I didn't check the Drupal and PHP version before working on
    the implementation of hook_requirement().
    providing primary logic for hook_requirement().

  • Status changed to Needs work over 1 year ago
  • 🇺🇸United States Cellar Door

    Thanks for your work on this! It is a great start.

    I think we should use the already in place logic for the cert location (using the stored config w/ default fallbacks) and possibly we can borrow the logic from the expiration warning on the admin page for levels of severity.

    Once we account for those I think this can be included in the 5.x release

  • Status changed to RTBC over 1 year ago
  • 🇺🇸United States Cellar Door

    Updated the function to only run on runtime phase (not install or update) so it will show in the site status and also to use the config determined location of the certificate. Should also now work with automated hosts like Pantheon who don't require the user to manage their certificates.

  • First commit to issue fork.
  • 🇨🇴Colombia jlockhart Nomadic: Colombia

    Reviewed and agree.

  • Assigned to Cellar Door
  • 🇨🇴Colombia jlockhart Nomadic: Colombia

    Assigned to Chris for acceptance.

  • Status changed to Fixed over 1 year ago
  • 🇺🇸United States Cellar Door

    Merged in - thanks for the work all!

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

Production build 0.71.5 2024