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

Update Django-Q to version that fixes the SSL errors #288

Merged
merged 1 commit into from
Jun 2, 2021

Conversation

jbothma
Copy link
Contributor

@jbothma jbothma commented Jun 2, 2021

Description

We've been getting lots of SSL error: decryption failed or bad record mac errors from async tasks.

This has been fixed in Koed00/django-q#556 which is included in 1.3.6

Related Issue

How to test it locally

Changelog

Added

Updated

Removed

Checklist

  • 🚀 is the code ready to be merged and go live?
  • 🛠 does it work (build) locally

Pull Request

  • 📰 good title
  • 📝good description
  • 🔖 issue linked
  • 📖 changelog filled out

Commits

  • commits are clean
  • commit messages are clean

Code Quality

  • 🚧 no commented out code
  • 🖨 no unnecessary logging
  • 🎱 no magic numbers
  • black was run locally (as part of the pre-commit hook)

Testing

  • ✅ added (appropriate) unit tests
  • 💢 edge cases in tests were considered
  • ✅ ran tests locally & are passing

@milafrerichs milafrerichs temporarily deployed to wazimap-fix-postgres-ss-02phyy June 2, 2021 11:40 Inactive
@milafrerichs milafrerichs changed the title Update Django-Q to versiont that fixes the SSL errors Update Django-Q to version that fixes the SSL errors Jun 2, 2021
@milafrerichs milafrerichs merged commit 35a9420 into staging Jun 2, 2021
@jbothma jbothma deleted the fix-postgres-ssl-errors branch June 7, 2021 13:48
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