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

add decrypt table (the reverse of encrypt table) #87

Merged
merged 7 commits into from Feb 27, 2015
Merged

add decrypt table (the reverse of encrypt table) #87

merged 7 commits into from Feb 27, 2015

Conversation

serixscorpio
Copy link
Contributor

This is a functionality I found useful. This is used as a reverse step of encrypt_table! in the context of calling in within a migration. The decrypt_table! method allows one to rollback the migration.

@serixscorpio
Copy link
Contributor Author

I took a look at the Travis CI build error, but was not able to figure out why that error occurred. It looks like starting postgresql server for build 528.9 timed out for some reason. https://travis-ci.org/jmazzi/crypt_keeper/jobs/39288632

@dankohn
Copy link
Contributor

dankohn commented Nov 19, 2014

@jmazzi, could you please rerun Travis on the pull request, as one build failed when Postgres wouldn't start. We would love to merge this so that we don't need to maintain our fork.

@jmazzi
Copy link
Owner

jmazzi commented Nov 19, 2014

@serixscorpio @dankohn i'm in the midst of a possible big refactor so I"m holding off on merging anything until I can see how it plays out. Sorry for the delay.

@acrogenesis
Copy link

any update on this merge? @jmazzi @serixscorpio

@jmazzi
Copy link
Owner

jmazzi commented Feb 22, 2015

@serixscorpio @dankohn @acrogenesis this PR needs a rebase and then I can merge.

@serixscorpio
Copy link
Contributor Author

Also, I noticed the crypt_keeper version in various gemfile.lock used by travis ci need to be updated from 0.18.4 to 0.19.0. 7ff84be

@jmazzi
Copy link
Owner

jmazzi commented Feb 26, 2015

@serixscorpio you need to run bundle exec appraisal install. That will update the files

@serixscorpio
Copy link
Contributor Author

I see. Thanks! Appraisal works great! I'm curious, what's the rationale behind keeping the different gemfile.lock in version control?

@jmazzi
Copy link
Owner

jmazzi commented Feb 27, 2015

@serixscorpio oversight :) - opened #97

jmazzi added a commit that referenced this pull request Feb 27, 2015
add decrypt table (the reverse of encrypt table)
@jmazzi jmazzi merged commit 1891041 into jmazzi:master Feb 27, 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.

None yet

4 participants