geraj1010 commented on code in PR #44780:
URL: https://github.com/apache/airflow/pull/44780#discussion_r1883209573


##########
airflow/www/static/js/main.js:
##########
@@ -286,4 +286,8 @@ $(document).ready(() => {
 
   // Global Tooltip selector
   $(".js-tooltip").tooltip();
+
+  // Turn off autocomplete for login form
+  $("#username:input")[0].autocomplete="off";

Review Comment:
   I was just trying to be consistent. The issue only mentions password, but I 
didn't think it would hurt to apply the change to username too. Then it's even 
more "secure" from the issue's perspective I think.
   
   Shall I apply this change on `main` and reopen the PR there and close this 
one?



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