Skip to content

Update webui.py with username and password #292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

natarajan0007
Copy link

During the demo, we may demonstrate parsing usernames and passwords, which are currently displayed in plain text in the UI using task descriptions. By implementing this change, users can select the credentials required for their website's login process. However, please note that this does not prevent sensitive data from appearing in the console logs. It's a good approach for getting started initially.

During the demo, we may demonstrate parsing usernames and passwords, which are currently displayed in plain text in the UI using task descriptions. By implementing this change, users can select the credentials required for their website's login process. However, please note that this does not prevent sensitive data from appearing in the console logs. It's a good approach for getting started initially.
@warmshao
Copy link
Collaborator

This is a bit too customized and not universal enough. For example, what should I do if I have multiple accounts and passwords? Or for other types of sensitive data, I think we need to come up with a more universal way.

@pksorensen
Copy link
Contributor

My solution solves the same problem #316

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants