Skip to content
This repository has been archived by the owner on Apr 1, 2023. It is now read-only.

Sidekiq::Status doesn't play well with Sidekiq Enterprise's encryption feature #153

Open
alecgorge opened this issue Oct 5, 2019 · 1 comment

Comments

@alecgorge
Copy link

When I call perform_async on an encrypted worker it hits the branch in api.rb (379 is a print line I added) that anonymizes things:

image

This caused the value [encrypted data] to be encrypted and stored in redis. When the job is run, it appropriately decrypts the data and passes that literal string "[encrypted data]" to the method.

This plugin calls display_args which then calls down into the base implementation of display_args.

Making my jobs that need encryption bypass calling include Sidekiq::Status::Worker has fixed the issue for me.

I don't know the middleware stack well enough to know if there is a fix for this or the "solution" is simply not using status workers but I thought I should file something to help others out.

@kenaniah
Copy link
Collaborator

kenaniah commented Oct 9, 2019

Have you tried adjusting the order of the middleware? It's not guaranteed, but by flipping the order, they might be able to play nicely with each other.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants