Trim comments and descriptions

Author: gtxu

providers/http/docs/connections/http.rst

@@ -47,8 +47,8 @@ Password (optional)
 Host (optional)
     Specify the entire url or the base of the url for the service.
 
-    If "Use DNS SRV Lookup" is enabled below, specify the DNS SRV record name instead
-    (e.g. ``_http._tcp.example.com``) — the actual host and port are resolved from DNS at
+    If "Use DNS SRV Lookup" is enabled, specify the DNS SRV record name instead
+    (e.g. ``_http._tcp.example.com``) - Note the actual host and port are resolved from DNS at
     request time and any value set in the Port field is ignored.
 
 Port (optional)
@@ -58,13 +58,10 @@ Schema (optional)
     Specify the service type etc: http/https.
 
 Use DNS SRV Lookup (optional)
-    When enabled, the Host field above is treated as a DNS SRV record name that is resolved
-    to the actual target host and port at request time, instead of connecting to Host/Port
-    directly. Requires the ``apache-airflow-providers-http[srv]`` extra (``dnspython``).
+    Treat the Host field as a DNS SRV record name and resolve the target host/port at request time.
 
 SRV Cache TTL (seconds) (optional)
-    How long to cache a resolved SRV target before re-resolving. Defaults to 60 seconds.
-    Only used when SRV lookup is enabled.
+    Specify the time to cache a resolved SRV target before re-resolving. (default 60 seconds)
 
 Extra (optional)
     Specify headers and default requests parameters in json format.

providers/http/provider.yaml

@@ -130,17 +130,16 @@ connection-types:
       srv_lookup:
         label: Use DNS SRV Lookup
         description: >-
-          When enabled, the Host field is treated as a DNS SRV record name (e.g.
-          _http._tcp.example.com) that is resolved to the actual target host and port at
-          request time. Requires the apache-airflow-providers-http[srv] extra.
+          Whether to treat the Host field as a DNS SRV record name and resolve the target
+          host/port at request time.
         schema:
           type:
             - boolean
             - "null"
           default: false
       srv_cache_ttl:
         label: SRV Cache TTL (seconds)
-        description: How long to cache a resolved SRV target before re-resolving.
+        description: Time to cache a resolved SRV target before re-resolving.
         schema:
           type:
             - number

providers/http/src/airflow/providers/http/get_provider_info.py

@@ -69,12 +69,12 @@ def get_provider_info():
                 "conn-fields": {
                     "srv_lookup": {
                         "label": "Use DNS SRV Lookup",
-                        "description": "When enabled, the Host field is treated as a DNS SRV record name (e.g. _http._tcp.example.com) that is resolved to the actual target host and port at request time. Requires the apache-airflow-providers-http[srv] extra.",
+                        "description": "Whether to treat the Host field as a DNS SRV record name and resolve the target host/port at request time.",
                         "schema": {"type": ["boolean", "null"], "default": False},
                     },
                     "srv_cache_ttl": {
                         "label": "SRV Cache TTL (seconds)",
-                        "description": "How long to cache a resolved SRV target before re-resolving.",
+                        "description": "Time to cache a resolved SRV target before re-resolving.",
                         "schema": {"type": ["number", "null"], "minimum": 0, "default": 60},
                     },
                 },

providers/http/src/airflow/providers/http/hooks/http.py

@@ -57,19 +57,14 @@ def _url_from_endpoint(base_url: str | None, endpoint: str | None) -> str:
 
 
 def _select_srv_target(answers: Iterable[Any]) -> tuple[str, int]:
-    """
-    Select a target host and port from a set of resolved DNS SRV records.
-
-    Among the records with the lowest (i.e. highest-priority) priority value, one is chosen
-    uniformly at random, following the failover guidance in RFC 2782. Shared between the sync
-    and async SRV resolution paths, which only differ in how they call the DNS resolver.
-    """
+    """Select a target host and port from resolved DNS SRV records."""
     candidates_by_priority: dict[int, list[Any]] = {}
     for record in answers:
         candidates_by_priority.setdefault(record.priority, []).append(record)
+    # Failover mechanism: RFC 2782
     candidates = candidates_by_priority[min(candidates_by_priority)]
     chosen = random.choice(candidates)
-    # SRV targets are FQDNs with a trailing dot; strip it for use as a hostname / Host header.
+
     target_host = str(chosen.target).rstrip(".")
     return target_host, chosen.port
 
@@ -158,14 +153,10 @@ class HttpHook(BaseHook):
     :param tcp_keep_alive_interval: The TCP Keep Alive interval parameter (corresponds to
         ``socket.TCP_KEEPINTVL``)
 
-    The connection's Extra field also supports resolving the target host via a DNS SRV record
-    instead of a fixed host/port, by setting:
+    Extra also supports resolving ``host`` via a DNS SRV record:
 
-    * ``srv_lookup`` (bool): When true, ``host`` is treated as a DNS SRV record name (e.g.
-      ``_http._tcp.example.com``) that is resolved to the actual target host and port at
-      request time. Requires the ``apache-airflow-providers-http[srv]`` extra (``dnspython``).
-    * ``srv_cache_ttl`` (float): How long, in seconds, to cache a resolved SRV target before
-      re-resolving. Defaults to 60 seconds.
+    * ``srv_lookup`` (bool): treat ``host`` as an SRV record name, e.g. ``_http._tcp.example.com``.
+    * ``srv_cache_ttl`` (float): SRV cache TTL in seconds (default 60).
     """
 
     conn_name_attr = "http_conn_id"
@@ -644,9 +635,10 @@ class HttpAsyncHook(BaseHook):
     :param retry_limit: Maximum number of times to retry this job if it fails  (default is 3)
     :param retry_delay: Delay between retry attempts (default is 1.0)
 
-    Supports the same ``srv_lookup`` / ``srv_cache_ttl`` Extra field options as ``HttpHook``
-    (see there for details) for resolving the target host via a DNS SRV record. Resolution is
-    performed with the non-blocking ``dns.asyncresolver``.
+    Extra also supports resolving ``host`` via a DNS SRV record:
+
+    * ``srv_lookup`` (bool): treat ``host`` as an SRV record name, e.g. ``_http._tcp.example.com``.
+    * ``srv_cache_ttl`` (float): SRV cache TTL in seconds (default 60).
     """
 
     conn_name_attr = "http_conn_id"