Skip to content
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

Support simultaneously PAC file and system credentials #13

Closed
CidTori opened this issue Mar 7, 2023 · 4 comments
Closed

Support simultaneously PAC file and system credentials #13

CidTori opened this issue Mar 7, 2023 · 4 comments
Labels
wontfix This will not be worked on

Comments

@CidTori
Copy link

CidTori commented Mar 7, 2023

Feature request's description
I need to use both a PAC file and system credentials (for security reasons) simultaneously.

The solution
I would like the same "Use system credentials" feature from the HTTP proxy type for the PAC proxy type.

@ecovaci
Copy link
Owner

ecovaci commented Mar 7, 2023

I don't think I understand your request. On Windows, if the Username/Password fields not filled in, the current credentials are used. Why do you need to provide both? How does this improve the security, when credentials in Winfoom are kept encoded but not encrypted?

@ecovaci ecovaci added the wontfix This will not be worked on label Mar 7, 2023
@CidTori
Copy link
Author

CidTori commented Mar 7, 2023

When using the GUI and choosing the PAC proxy type, there is no "Use system credentials" checkbox, contrary to the HTTP proxy type, which means, I suppose, that using system credentials is not supported with the PAC proxy type, which means that, since I need to use a PAC file, I have to put my username and password, in plain text, in the configuration file, which isn't great in terms of security.

@ecovaci
Copy link
Owner

ecovaci commented Mar 8, 2023

That's wrong. In PAC mode, when no credentials are provided, the system credentials are used. Read the tooltip of the Username/Password fields in PAC mode

@ecovaci ecovaci closed this as completed Mar 8, 2023
@CidTori
Copy link
Author

CidTori commented Mar 8, 2023

You are right, I didn't see the tooltip. In that case, why not display the same checkbox as with the HTTP proxy type? It would be harder to miss than the tooltip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants