Skip to content

Update redis-session-store to fork with security fixes (LG-5369)#5609

Merged
zachmargolis merged 1 commit intomainfrom
margolis-redis-session-store-patch
Nov 15, 2021
Merged

Update redis-session-store to fork with security fixes (LG-5369)#5609
zachmargolis merged 1 commit intomainfrom
margolis-redis-session-store-patch

Conversation

@zachmargolis
Copy link
Contributor

See: 18F/redis-session-store#5

When roidrage/redis-session-store#125 is merged, we'll be able to go back to the rubygems version

gem 'redis', '>= 3.2.0'
gem 'redis-namespace'
gem 'redis-session-store', '>= 0.11.3'
gem 'redis-session-store', github: '18f/redis-session-store', tag: 'v0.11.4-18f'
Copy link
Contributor Author

@zachmargolis zachmargolis Nov 15, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know we have the pattern up above with @hostdata_gem ||= to allow for local development, but we really don't iterate with this gem often so I figured it was easier to use the simpler form like this

@zachmargolis zachmargolis merged commit 3839771 into main Nov 15, 2021
@zachmargolis zachmargolis deleted the margolis-redis-session-store-patch branch November 15, 2021 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants