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

DB password #389

Closed
wants to merge 288 commits into from
Closed

DB password #389

wants to merge 288 commits into from

Conversation

wilbertom
Copy link
Member

This commit forces a change in the database password and moves it to the environment. closes #386

nwinklareth and others added 30 commits October 9, 2013 23:36
V2.0 dev -- prepped a v2.0 flask app with gunicorn script, nginx conf, and simplified routing
Fixed paths to new 2.0 structure
Added logic to record current and new version build ids.
…e active website link to the newly installed website.
This correctly makes a release based on git commit ids, and points the a...
JSONified static return values
Turned off caching for development version.
Added route to display os environment info, first info in current workin...
Added fetching current commit id and deployed date and time.
Added explanation of how to use Fabric tool to deploy and roll back webs...
@wilbertom
Copy link
Member Author

Ups, looks like some api code made it there too. Darn it.

@bepetersn
Copy link
Member

You made an interesting function to prompt people to set env bars, but not
the important work of setting them in any of our admin files, like
scraper.sh, so this will make the scraper fail.
On May 19, 2014 4:07 PM, "Wilberto Morales" [email protected]
wrote:

Ups, looks like some api code made it there too. Darn it.


Reply to this email directly or view it on GitHubhttps://github.com//pull/389#issuecomment-43558147
.

@wilbertom
Copy link
Member Author

Yup, I have no idea where I gotta tweak those env variables. I'll add it in a few minutes.

@bepetersn
Copy link
Member

I'm still setting up my system, or I'd do it.

@bepetersn
Copy link
Member

hmmmmmmmm, well I messed up that pull request; @wilbertom, can you re-do it?

@bepetersn bepetersn closed this May 20, 2014
@wilbertom
Copy link
Member Author

Hey Brian, no problem, I still needed to add it to the server's env.

@wilbertom
Copy link
Member Author

Wait how did the password get to the deployed app the first time? I don't see any code to tweak. Did someone just ssh and created the database? Or was the database created through an admin interface? If so then it might be easier to use one of those methods again.

~/Programming/SC3/cookcountyjail (db-password) wil$ git grep 'walblgadb;lgall'
ccj/config.py:    if db_config['pw'] == 'walblgadb;lgall':

~/Programming/SC3/cookcountyjail (db-password) wil$ git grep cookcountyjail_v2.0_dev
ccj/config.py:        'db_name': env_var('CCJ_DBNAME', 'cookcountyjail_v2_0_dev'),
tests/test_pgsql_installation.py:    result = local('psql cookcountyjail_v2_0_dev -c "\dt"', capture=True)
tests/test_pgsql_installation.py:    local('psql cookcountyjail_v2_0_dev -c "DROP TABLE elaborate_test;"', capture=True)

@bepetersn
Copy link
Member

Well, we should try and handle this the right way.

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.

3 participants