Skip to content

Integration of Proxy Vole#21

Open
serge-xav wants to merge 6 commits into
bekoenig:masterfrom
serge-xav:proxyvole
Open

Integration of Proxy Vole#21
serge-xav wants to merge 6 commits into
bekoenig:masterfrom
serge-xav:proxyvole

Conversation

@serge-xav

Copy link
Copy Markdown
Contributor

Integration of ProxyVole proposal.

Current behavior kept. Use the system properties use_proxy=proxy_vole to enable this new behavior.

@serge-xav

Copy link
Copy Markdown
Contributor Author

I've changed my mind about current behavior for proxy search. By default proxy vole will be used, also if use_proxy system property is set to true. If set to "false", no proxy detection done. If set to "legacy", usage of old JRegistryKey way

@bekoenig

bekoenig commented Jul 8, 2025

Copy link
Copy Markdown
Owner

Hello @serge-xav,

thanks for your improvement. The changes look good, but I'm a bit wary of ProxyVole. This project is outdated and it needs a lot of dependencies. Junit has compile scope, so I'm very skeptical.

@bekoenig bekoenig linked an issue Jul 9, 2025 that may be closed by this pull request
@serge-xav

Copy link
Copy Markdown
Contributor Author

Hi @bekoenig thanks for your reply, I've not seen the notification.

Understood your point on dependencies, but outdated is more valid for JRegistryKey than for Proxy-Vole I would say.

And as I said, JRegistryKey is of no use if you have a 64bit JVM running getdown.

@bekoenig

bekoenig commented Oct 6, 2025

Copy link
Copy Markdown
Owner

Hi @serge-xav ,

I have to reject your PR. Proxy Vole is dead and has a lot of vulnerables (see here).

Inlining the code is also no option for me, because I don't use windows and have no experience in proxy detection. So I can't support any Issues for this implementation.

Many thanks for sharing your work.

Greetings,
Ben

@bekoenig bekoenig closed this Oct 6, 2025
@bekoenig bekoenig reopened this Oct 7, 2025
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.

Get rid of JRegistryKey for Proxy discover, use Proxy Vole instead

2 participants