[GitHub] [airflow] eladkal commented on a change in pull request #16916: Deprecate Tableau personal token authentication

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

eladkal commented on a change in pull request #16916:
URL: https://github.com/apache/airflow/pull/16916#discussion_r667366994



##########
File path: airflow/providers/tableau/hooks/tableau.py
##########
@@ -41,6 +40,16 @@ class TableauHook(BaseHook):
     """
     Connects to the Tableau Server Instance and allows to communicate with it.
 
+    Can be used as a context manager: automatically authenticates the connection
+    when opened and signs out when closed.
+
+    WARNING: authenticaton by personal token was deprecated as Tableau automatically
+    invalidates opened personal token connection if one or more parallel
+    connections with the same token are opened. So, in the environments with
+    multiple parallel tasks this authentication method can lead to numerous bugs
+    and all the jobs will not run as they intended. Therefore personal token
+    auth option is no more available for the end user.

Review comment:
       The place for this is in the docs:
   https://github.com/apache/airflow/blob/main/docs/apache-airflow-providers-tableau/connections/tableau.rst




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org