Configuration data was added to $options so it would be available in an encryption method plugin's encryption callback function (see Make configuration available in the encryption callback function). This works OK, but it would be better if the data were made available as a separate argument.

CommentFileSizeAuthor
#1 encrypt-pass-config-2401681-0.patch1000 bytesrlhawk

Comments

rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
commented
Status: Active » Needs review
StatusFileSize
new encrypt-pass-config-2401681-0.patch1000 bytes

This patch undoes the change in issue #2371789 and adds $config as an argument to the callback function. Encryption method plugins can take advantage of its availability.

cellar door’s picture

cellar door commented
Status: Needs review » Reviewed & tested by the community

Simple one-liner but allows encryption plugin's to know more about the scenario they are encrypting for (necessary if there are settings etc. that need to get passed in along with the configuration like key provider etc.

I've used it in testing an encryption plugin and it's working great. Moving to RTBC

  • rlhawk committed e1b86b0 on 7.x-2.x 
    Issue #2401681 by rlhawk: Pass configuration to encryption method...
rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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