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

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2023/01/04 15:26:19 UTC

[GitHub] [airflow] potiuk commented on pull request #28730: Update CSRF token to expire with session

potiuk commented on PR #28730:
URL: https://github.com/apache/airflow/pull/28730#issuecomment-1371070850

   > Does this introduce any problem/risk of CSRF token re-use?
   
   Not really. This is recommended setting fos CSRF. I read quite a bit about CSRF when https://github.com/apache/airflow/issues/28356 was raised and I agree expiry with session is ok. CSRF token is not to authorize the session (this is already done via session token). The only role of CSRF is to protect against the case that any "static" URL generated outside the main page will be seen as coming from the application. 


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