njnu-seafish commented on PR #18386:
URL: 
https://github.com/apache/dolphinscheduler/pull/18386#issuecomment-4832130311

   > I think this is unnessnary since we've already return hiddened password. 
There is no essential difference between your implementation and the current 
one.
   
   In the current implementation, saving without changing the password can 
easily lead to mistakenly treating the hidden password returned by the backend 
as the actual password in the data source.   
   
   Based on the feedback in issues like 
https://github.com/apache/dolphinscheduler/issues/17813 and 
https://github.com/apache/dolphinscheduler/issues/18004, users are currently 
prone to accidentally overwriting the correct password due to the mask.
   
   This PR implements the solution discussed previously by the senior team: the 
backend no longer returns the password, and the frontend prompts the user to 
'Enter a new password (leave blank to keep the current 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