You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "jason810496 (via GitHub)" <gi...@apache.org> on 2025/02/28 04:05:00 UTC

Re: [PR] AIP-84 | Add Auth for Dags [airflow]

jason810496 commented on PR #47062:
URL: https://github.com/apache/airflow/pull/47062#issuecomment-2689666179

   
   After discussing with @rawwar offline, we determined that the previous test failure was caused by JWT and test cases using `time_machine`.  
   
   > The test cases with `time_machine` are decorated with `@time_machine.travel(timezone.utcnow(), tick=False)`. However, when the method is executed, the `test_client` fixture is recreated, causing the token to be perceived as being from the future.  
   
   After further discussion, by setting `iat` and `nbf` to a much earlier time (when Airflow was created) and `exp` to 24 hours in the future resolves the issue.  
   
   > - **`iat`**: Issued at  
   > - **`nbf`**: Not before  
   > - **`exp`**: Expiration time  
   
   cc @pierrejeambrun 
   
   


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