[GitHub] [airflow] nuclearpinguin commented on a change in pull request #7205: [AIRFLOW-XXXX] Adjust celery defaults to work with breeze

[GitBox <gi...@apache.org>]

nuclearpinguin commented on a change in pull request #7205: [AIRFLOW-XXXX] Adjust celery defaults to work with breeze
URL: https://github.com/apache/airflow/pull/7205#discussion_r368231017
 
 

 ##########
 File path: airflow/config_templates/default_celery.py
 ##########
 @@ -40,7 +40,7 @@ def _broker_supports_visibility_timeout(url):
         broker_transport_options['visibility_timeout'] = 21600
 
 DEFAULT_CELERY_CONFIG = {
-    'accept_content': ['json', 'pickle'],
+    'accept_content': ['json'],
 
 Review comment:
   Reason for this changes:
   ```
   Running a worker with superuser privileges when the
   worker accepts messages serialized with pickle is a very bad idea!
   
   If you really want to continue then you have to set the C_FORCE_ROOT
   environment variable (but please think about this before you do).
   
   User information: uid=0 euid=0 gid=0 egid=0
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services