SameerMesiah97 opened a new pull request, #62643:
URL: https://github.com/apache/airflow/pull/62643

   **Description**
   
   Refactored `VaultBackend` to eliminate duplicated secret path resolution and 
secret retrieval logic across `get_response`, `get_variable`, and `get_config`. 
These methods previously repeated mount parsing, base path handling, path 
construction, and invocation of `_VaultClient.get_secret`.
   
   This change introduces a private helper that centralizes path resolution and 
secret fetching, while preserving the existing post-processing behavior in each 
public method.
   
   **Rationale**
   
   The previous implementation duplicated non-trivial mount parsing, path 
construction, and secret retrieval logic in three separate methods. 
Centralizing this logic reduces maintenance risk and prevents divergence 
between connection, variable, and config resolution paths. This refactor 
improves internal structure without altering semantics.
   
   **Tests**
   
   No new tests were added. Existing tests already validate mount handling, 
path resolution, and secret retrieval behavior.
   
   **Backwards Compatibility**
   
   No changes to public APIs or behavior. This is an internal refactor only.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to