Hi,

When a code path requests a reset on the default config entry through encrypt_get_default_config(TRUE), it is getting a stale version if such is already stored in encrypt_get_configs static cache. This results in the usage of stale/wrong configuration at that moment.

The fix is very simple, just pass the reset variable through, so it will propagate to the places needed.
I will provide a patch tomorrow morning.

CommentFileSizeAuthor
#2 encrypt-encrypt_get_default_config-TRUE-does-not-reset-config-2632668-2.patch500 bytesndobromirov
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ndobromirov created an issue. See original summary.

ndobromirov’s picture

ndobromirov CreditAttribution: ndobromirov at FFW commented
Assigned: ndobromirov » Unassigned
Issue summary: View changes
Status: Active » Needs review
FileSize
encrypt-encrypt_get_default_config-TRUE-does-not-reset-config-2632668-2.patch500 bytes

Here is a patch that fixes the issue for me.
Small tweak to the issue description.

Br. Nikolay Dobromirov.

rlhawk’s picture

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

Looks good.

  • rlhawk committed 66bb2bd on 7.x-2.x authored by ndobromirov 
    Issue #2632668 by ndobromirov: encrypt_get_default_config should...
rlhawk’s picture

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

Committed.

Status: Fixed » Closed (fixed)

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