[
https://issues.apache.org/jira/browse/AIRFLOW-3127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16631564#comment-16631564
]
ASF GitHub Bot commented on AIRFLOW-3127:
-----------------------------------------
ashb closed pull request #3967: [AIRFLOW-3127] Fix out-dated doc for Celery SSL
URL: https://github.com/apache/incubator-airflow/pull/3967
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/docs/security.rst b/docs/security.rst
index 60fe160404..89b5605daa 100644
--- a/docs/security.rst
+++ b/docs/security.rst
@@ -342,10 +342,10 @@ certs and keys.
.. code-block:: bash
[celery]
- CELERY_SSL_ACTIVE = True
- CELERY_SSL_KEY = <path to key>
- CELERY_SSL_CERT = <path to cert>
- CELERY_SSL_CACERT = <path to cacert>
+ ssl_active = True
+ ssl_key = <path to key>
+ ssl_cert = <path to cert>
+ ssl_cacert = <path to cacert>
Impersonation
-------------
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Celery SSL Documentation is out-dated
> -------------------------------------
>
> Key: AIRFLOW-3127
> URL: https://issues.apache.org/jira/browse/AIRFLOW-3127
> Project: Apache Airflow
> Issue Type: Improvement
> Components: Documentation
> Affects Versions: 1.10.0
> Reporter: Xiaodong DENG
> Assignee: Xiaodong DENG
> Priority: Minor
>
> Now in `airflow.cfg`, for Celery-SSL, the items names are "ssl_active",
> "ssl_key", "ssl_cert", and "ssl_cacert". (since PR
> https://github.com/apache/incubator-airflow/pull/2806/files)
> But in the documentation
> [https://airflow.incubator.apache.org/security.html?highlight=celery]
> or
> [https://github.com/apache/incubator-airflow/blob/master/docs/security.rst,]
> it's "CELERY_SSL_ACTIVE", "CELERY_SSL_KEY", "CELERY_SSL_CERT", and
> "CELERY_SSL_CACERT", which is out-dated.
>
> This may confuse readers.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
