-
Notifications
You must be signed in to change notification settings - Fork 381
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
cannot use http://app as a baseurl #262
Comments
Not sure why it would be the case, can you try renaming the container from |
nope project is pretty old and big, and there is a lot of hardcoded hostnames, and right now we are trying to implement e2e testing should it not be easy to just provide in cypress option like also its clear that problem happened from 3.8.1 to 3.8.2, should it be easy to track it? |
Did you try to add the port of your app like this
|
my app must work on 80 port, because its the entrypoint of the whole app |
This comment has been minimized.
This comment has been minimized.
As I stated in description it started to happen in 3.8.2 |
That's the version included in https://github.com/textbook/cypress-test-tiny/tree/recreate-base-url-issue |
give me a hour - i'll create another repro repo |
https://github.com/Hulkmaster/cypress-app-test the strange part is doesn't work on 4.5.0 |
Yeah, I can recreate the issue as provided from this repo running
|
I updated the reproduction to 12.3.0. Works fine now. |
So i'm using
cypress/included
image, and recently i tried to upgrade and started to have an issuethat issue started to happen after upgrading
from
cypress/included:3.8.1
to
cypress/included:3.8.2
also might be related cypress-io/cypress#5482 (there you can also find repro)
thats my config
so on
cypress/included:3.8.1
works finestarting
cypress/included:3.8.2
have an error (also on 4.0.2)UPD:
new test repo https://github.com/Hulkmaster/cypress-app-test
works on 3.8.1
works on 3.8.2
doesn't work on 4.5.0
The text was updated successfully, but these errors were encountered: