Merge pull request #331 from Seluj78/hotfix/fixed-celery

Updated Celery config

Author: Seluj78

backend/PyMatcha/__init__.py

@@ -107,8 +107,8 @@
 
 logging.debug("Configuring Celery Redis URLs")
 # Celery configuration
-application.config["CELERY_BROKER_URL"] = CELERY_BROKER_URL
-application.config["CELERY_RESULT_BACKEND"] = CELERY_RESULT_BACKEND
+application.config["CELERY_broker_url"] = CELERY_BROKER_URL
+application.config["result_backend"] = CELERY_RESULT_BACKEND
 
 logging.debug("Initializing Celery")
 # Initialize Celery

backend/PyMatcha/routes/api/profile/edit.py

@@ -185,7 +185,7 @@ def edit_email():
     token = generate_confirmation_token(email=new_email, token_type="confirm")
     link = FRONTEND_EMAIL_CONFIRMATION_URL + token
     rendered_html = render_template("confirm_email.html", link=link)
-    send_mail_html.delay(dest=data["email"], subject="Confirm your email on PyMatcha", html=rendered_html)
+    send_mail_html.delay(dest=new_email, subject="Confirm your email on PyMatcha", html=rendered_html)
     return Success("Email sent for new email")
 
 

backend/requirements.txt

@@ -20,7 +20,7 @@ cryptography==3.1.1
 
 flask_jwt_extended==3.24.1
 
-flake8==3.8.3
+flake8==3.8.4
 black==20.8b1
 pre-commit==2.7.1
 

workers.Dockerfile

@@ -7,4 +7,4 @@ ENV PYTHONDONTWRITEBYTECODE 1
 EXPOSE 5000
 ADD .env .
 RUN export $(cat .env | xargs)
-CMD celery worker --workdir . -A PyMatcha.celery -B --loglevel=info --concurrency=1
+CMD celery -A PyMatcha.celery worker --concurrency=1 -E --loglevel=INFO -B