fecgov / fec-eregs

The Federal Election Commission's web-based application that makes regulations easier to find, read and understand.
https://www.fec.gov/regulations/
Other
33 stars 13 forks source link

upgrade django, cryptography, prompt-toolkit, and ipython #752

Closed tmpayton closed 1 year ago

tmpayton commented 1 year ago

Summary (required)

This ticket fixes the snyk vulnerbilities in requirements-parsing.txt and upgrades django in requirements.txt.

packages upgraded:

Required reviewers 2 developers

Impacted areas of the application

General components of the application that this PR will affect:

Screenshots

Before:

requirements.txt:

image

requirements-parsing.txt: image

After:

requirements.txt: image

requirements-parsing.txt: image

Related PRs

Related PRs against other branches:

reg-core PR reg-parser PR reg-site PR

How to test

  1. Checkout this branch
  2. Change regparser, regsite, and regcore to point to my branch in requirements.txt and requirements-parsing.txt

-e git+https://github.com/fecgov/regulations-parser.git@update-requirements-parsing#egg=regparser

-e git+https://github.com/fecgov/regulations-site@update-requirements-parsing#egg=regulations

-e git+https://github.com/fecgov/regulations-core@update-requirements-parsing#egg=regcore

Terminal One:

  1. pyenv virtualenv (your virtual environment)
  2. pip install -r requirements.txt
  3. snyk test --file=requirements.txt --package-manager=pip
  4. rm -rf node_modules
  5. npm i
  6. npm run build
  7. dropdb eregs_local
  8. createdb eregs_local
  9. python manage.py migrate
  10. python manage.py compile_frontend
  11. python manage.py runserver

Terminal Two:

  1. pyenv virtualenv (your virtual environment)
  2. pip install -r requirements-parsing.txt
  3. snyk test --file=requirements-parsing.txt --package-manager=pip
  4. python load_regs/load_fec_regs.py local
  5. Go to http://127.0.0.1:8000/ to view 45 regulations

For more detailed instructions follow the wiki on how to setup/parse regulations on local environment

This will not pass circle build until other PRs are merged first.