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

Proxy settings are ignored without warning when connected to localhost #7169

Closed
HanaGemela opened this issue Apr 30, 2019 · 6 comments
Closed

Comments

@HanaGemela
Copy link
Contributor

Client: 2.6.0-daily20190429 (build 11690)
macOS 10.14.4

Steps to recreate:

  1. Run ownCloud server and client on your machine
  2. Connect the client to the server using localhost:8080
  3. Set up the proxy in the Network tab in the client

Actual result: User can set proxy but this settings is ignored
Expected result: Logs mention that proxy settings has been ignored. Also user interface warns that proxy is not used - e.g. if connected to localhost and proxy has been set up, the Account tab displays 'Connected to http://localhost:8080 as admin (no proxy)'

@HanaGemela HanaGemela added this to the 2.6.1 milestone Apr 30, 2019
@ckamm
Copy link
Contributor

ckamm commented May 2, 2019

Indeed, proxys are not used for local host by Qt (see https://doc.qt.io/qt-5/qnetworkproxy.html#details). The client could notify about that.

@ogoffart
Copy link
Contributor

ogoffart commented Jun 4, 2019

How should we notify about it? A label in the networksetting with a warning that is shown if at least one account has a localhost url?

@guruz
Copy link
Contributor

guruz commented Jun 4, 2019 via email

@butonic
Copy link
Member

butonic commented Sep 10, 2019

um what is the rationale behind this? if I want to debug connections I locally run mitmproxy as well as my apache .... but how do I make the client use mitmproxy?

hm ok I had expected to make it use the proxy by giving it the public ip of my machine... that did not work ... but using the public ip for the oc server did ...

😕 ... anyway ... workaround is good enough ... maybe list this as a warning when running owncloudcmd...

@ogoffart
Copy link
Contributor

um what is the rationale behind this?

I don't know. That's what the network library we are using (Qt) is doing.

I guess the rationale is maybe that network settings are used globally for things that need access to the internet, and localhost does not need that.

ogoffart added a commit that referenced this issue Oct 21, 2019
…localhost

Only show this if at least one account is detected to have an url that looks
like localhost, because this could otherwise be confusing

Issue #7169
ogoffart added a commit that referenced this issue Nov 18, 2019
…localhost

Only show this if at least one account is detected to have an url that looks
like localhost, because this could otherwise be confusing

Issue #7169
@ogoffart
Copy link
Contributor

A warning was added when localhost is used as an host.
But we can't apply the proxy for localhost because Qt does not allow it.

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

No branches or pull requests

5 participants