[GitHub] [airflow] AbhiPrasad commented on a change in pull request #8911: Support extra config options for Sentry

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

AbhiPrasad commented on a change in pull request #8911:
URL: https://github.com/apache/airflow/pull/8911#discussion_r445491051



##########
File path: docs/errors.rst
##########
@@ -39,6 +39,8 @@ Add your ``SENTRY_DSN`` to your configuration file e.g. ``airflow.cfg`` under ``
 .. note::
     If this value is not provided, the SDK will try to read it from the ``SENTRY_DSN`` environment variable.
 
+You can supply `additional configuration options <https://docs.sentry.io/error-reporting/configuration/?platform=python>`__ based on the Python platform via [sentry] section.

Review comment:
       Can you also quickly note the unsupported options here, just so everything is clear?
   
   ```
   UNSUPPORTED_SENTRY_OPTIONS = frozenset(
           ("integrations", "in_app_include", "in_app_exclude", "ignore_errors",
            "before_breadcrumb", "before_send", "transport")
       )
   ```




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