Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use GCM instead of CBC #167

Merged
merged 2 commits into from
Oct 29, 2015
Merged

use GCM instead of CBC #167

merged 2 commits into from
Oct 29, 2015

Conversation

gtank
Copy link
Contributor

@gtank gtank commented Oct 29, 2015

This strengthens the cryptography used to protect token-signing keys. Fixes #98

@bobbyrullo
Copy link
Contributor

pretty good, just a few little things

@bobbyrullo
Copy link
Contributor

LGTM

bobbyrullo added a commit that referenced this pull request Oct 29, 2015
@bobbyrullo bobbyrullo merged commit 9172f54 into dexidp:master Oct 29, 2015
bcwaldon pushed a commit to bcwaldon/dex that referenced this pull request Dec 10, 2015
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