Key integration

Created on 12 December 2024, 2 months ago

It would be nice to have some way to store sensitive keys, such as using a key module or other built-in approach. Initially, I saw that there is a related issue Release version Closed: won't fix with a patch of such functionality.
So I have adjusted the patch for the 2.x version of the module.

Feature request
Status

Active

Version

2.0

Component

Code

Created by

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

Merge Requests

Comments & Activities

  • Issue created by @Denist3r
  • 🇮🇹Italy apaderno Brescia, 🇮🇹

    Thank you for providing a patch! Since patches are no longer tested, we need a merge request.

    -        "drupal/mailsystem": "It provides an administrative UI and developers API for managing the used mail backend/plugin."
    +        "drupal/mailsystem": "It provides an administrative UI and developers API for managing the used mail backend/plugin.",
    +        "drupal/key": "^1.19"

    As with the existing suggestion, the content of the key is a description of the package or a description of why the package is suggested, not the package version to use.

  • 🇮🇳India shivam_tiwari

    I am working here.

  • Merge request !42Key integration with graph mail. → (Open) created by shivam_tiwari
  • Pipeline finished with Success
    about 2 months ago
    Total: 151s
    #381876
  • 🇮🇳India shivam_tiwari

    Created MR 42 for 2.0.x-dev branch.

  • 🇮🇹Italy apaderno Brescia, 🇮🇹
  • Pipeline finished with Failed
    about 2 months ago
    Total: 158s
    #382668
  • Pipeline finished with Success
    about 2 months ago
    Total: 146s
    #387076
  • Pipeline finished with Success
    about 2 months ago
    Total: 146s
    #387107
  • Status changed to RTBC 6 days ago
  • 🇮🇳India sayan_k_dutta

    Tested the MR. The key integration with the Graph mail module seems to work fine.
    Created a key of type Graph Mail and configured the graph api mail settings to work to with the key created. Used dummy values for testing. All the comments on the MR also seems to be addressed. Attaching a video for reference and moving it to RTBC.

  • 🇮🇹Italy apaderno Brescia, 🇮🇹
  • 🇮🇹Italy apaderno Brescia, 🇮🇹
  • 🇮🇹Italy apaderno Brescia, 🇮🇹

    Actually, since there are 59 sites using the 2.0.0-alpha7 release, I would prefer the changes are backward compatible, which means the Key module cannot be a hard dependency, or those sites will not be able to update.
    I know that alpha releases do not need to be backward compatible, but I would rather avoid disruptions on those sites.

    My idea was to wait to create a pre-release, to avoid such changes, but the pre-release was created, and we cannot turn back.

Production build 0.71.5 2024