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

Security configuration with redis support. #127

Merged
merged 16 commits into from
Sep 20, 2017

Conversation

wtrocki
Copy link
Member

@wtrocki wtrocki commented Sep 19, 2017

Motivation

Add redis to hold session.

Progress

  • Moved critical passport.js JWT token implementation that was
    interacting with session config (required for documentation)
  • Moved to use users as JWT payload (minor change with huge production implications)
  • Refactored config - to improve our documentation (minimize sets of steps for keycloak)
  • Included redis session into the code.
  • Configured session to be based on redis.
  • Improve coverage and unit tests spies.

Additional Notes

Steps to verify:

Keycloak and passport separately

  1. Login to portal application
  2. Restart server
  3. Go back. Session should still be valid and user should be logged in.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 5506cea on wtrocki:RAINCATCH-1176 into ** on feedhenry-raincatcher:master**.

1 similar comment
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 5506cea on wtrocki:RAINCATCH-1176 into ** on feedhenry-raincatcher:master**.

Copy link
Contributor

@JameelB JameelB left a comment

Choose a reason for hiding this comment

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

Verified locally

@wtrocki wtrocki merged commit fbaebc3 into feedhenry-raincatcher:master Sep 20, 2017
@wtrocki wtrocki deleted the RAINCATCH-1176 branch September 20, 2017 16:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

4 participants