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

Update testcase to emulate corporate proxy #2895

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

Conversation

Torbjorn-Svensson
Copy link
Contributor

The testcase wrongly assumed that the client can resolve the domain name.
Some corporate networks with proxy have internal name servers that does not include public records and this, resolving any non-internal domain names will fail. The proxy servers are having one leg in the real world and can thus resolve public records.

I've only updated the test case as I'm unable to build of tycho for some unknown reason.

To validate that the testcase did the intended, I had to:

  1. Remove every module in /pom.xml except tycho-its and tycho-testing-harness.
  2. Remove every java file in /tycho-its/src/test/java/ except /tycho-its/src/test/java/org/eclipse/tycho/test/TYCHO279HttpProxy/ProxySupportTest.java.
  3. Build using mvn -Dtycho-build-version=4.0.3 clean package.

Hope this fulfills the request for help in #2709 (comment).

Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@st.com>
@laeubi
Copy link
Member

laeubi commented Oct 7, 2023

To build/test you usually do:

mvn clean install -DskipTests -T1C

then cd into tycho-its then call

mvn clean install -Pits -Dtest=<classname of your test to run>

@laeubi
Copy link
Member

laeubi commented Oct 7, 2023

The testcase wrongly assumed that the client can resolve the domain name.

Why is this wrong? I'm not sure if such thing can ever work in a unit test reliable as name service is different from proxy service.

So if you like to have that we either also need to mock a DNS server or one needs to configure a mirror or similar.

@Torbjorn-Svensson
Copy link
Contributor Author

The testcase wrongly assumed that the client can resolve the domain name.

Why is this wrong? I'm not sure if such thing can ever work in a unit test reliable as name service is different from proxy service.

So if you like to have that we either also need to mock a DNS server or one needs to configure a mirror or similar.

Well, if you want the real deal, then no, it would not work, but I'm using a different approach.
In the test, I instead of stating the web-server that has the content and let the proxy just tag along, I define a different URL that the proxy knows that it should replace the the real URL. This way, I'm forcing the request to go though the proxy in order to actually resolve to anything, meaning that I emulate that the client cannot do the resolving while the proxy server "can" (obviously it can't but it instead replaces the request).
Does this make sense?

client
  -> request http://fake-non-existing-domain/random/path/ from proxy

proxy 
  -> get the request
  -> rewrite URL to http://localhost:12345/test/
  -> send the request of http://localhost:12345/test/
  -> forward the reply back to "client" as data from http://fake-non-existing-domain/random/path/

Obviously, fake-non-existing-domain can be anything that the client cannot access without the proxy. In my case, I assume that there is nothing that matches this string.

Do you think this way to emulate the scenario with a corporate network that requires to go thought a proxy to get out is valid?

Now that I'm thinking about this a 2nd time, I might have missed to replace all links to http://localhost:... with the http://fake-non... part in the reply.

@laeubi
Copy link
Member

laeubi commented Oct 8, 2023

Alright, but now the test fails with a message that proxy was not used while I would expect that it should fail with http://fake-non-existing-domain/random/path/ not found instead ...

@laeubi
Copy link
Member

laeubi commented Jan 6, 2024

@Torbjorn-Svensson there was a fix in the proxy support recently do you want to rebase your change and see if it fixes your issue?

@laeubi
Copy link
Member

laeubi commented Jan 23, 2024

@Torbjorn-Svensson can you rebase and resolve conflicts?

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.

None yet

2 participants