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

Dependencies update #102

Merged
merged 2 commits into from
Jun 17, 2021
Merged

Dependencies update #102

merged 2 commits into from
Jun 17, 2021

Conversation

krysal
Copy link
Member

@krysal krysal commented Jun 11, 2021

This PR compress all dependencies updates signaled by dependabot in order to address potential security vulnerabilities. Packages updated are:

  • django
  • django-redis
  • pytest-django
  • boto3
  • hvac
  • among others...

I also added the cache service as dependency of the web service in the docker-compose.yml file, because it's required by the application. This way we can run docker-compose up web without errors, omitting the rest of services (indexers, analytics, ...) it's useful for quick checks.

@krysal krysal requested review from obulat and dhruvkb June 11, 2021 16:24
@krysal krysal requested a review from a team as a code owner June 11, 2021 16:24
Copy link
Contributor

@obulat obulat left a comment

Choose a reason for hiding this comment

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

Builds fine, tests pass :)

Copy link
Member

@dhruvkb dhruvkb left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -63,6 +63,7 @@ services:
depends_on:
- db
- es
- cache
Copy link
Member

Choose a reason for hiding this comment

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

Nice catch!

@krysal krysal merged commit 7e468d7 into main Jun 17, 2021
@krysal krysal deleted the dependencies_update branch August 19, 2021 21:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants