Problem/Motivation

The module has no .install file. There is no installation-time validation that the authentication tokens have been configured (the shipped values are REPLACE_WITH_SECURE_... placeholders), and uninstalling leaves configuration behind.

Proposed resolution

  • Implement hook_requirements() warning when placeholder tokens or an insecure hash algorithm are still in use.
  • Implement hook_uninstall() to remove module configuration and state entries.
  • Add a status report entry summarising the API configuration.

Remaining tasks

Create the install file and cover it with a kernel test.

User interface changes

New entries on the Status report page.

API changes

None

Issue fork myrest-3615511

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

sergeydruua created an issue. See original summary.

sergeydruua opened merge request !47

  • sergeydruua committed f6fcfe84 on 1.0.x 
    Issue #3615511: Add runtime requirements and uninstall hooks to myrest....
sergeydruua’s picture

sergeydruua commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

sergeydruua’s picture

sergeydruua commented
Status: Fixed » Closed (fixed)