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.

Sign up for GitHub

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

Jump to bottom

Page without slash after location's host reloads if we set only hash #1426

Closed
LavrovArtem opened this issue Dec 15, 2017 · 2 comments
Closed

Page without slash after location's host reloads if we set only hash #1426

LavrovArtem opened this issue Dec 15, 2017 · 2 comments
Assignees
@LavrovArtem @Farfurix
Labels
AREA: client STATE: Auto-locked Issues that were automatically locked by the Lock bot SYSTEM: URL processing TYPE: bug
Milestone
Sprint #10

Comments

@LavrovArtem
Copy link
Contributor

see DevExpress/testcafe#2005
@LavrovArtem LavrovArtem added this to the Sprint #10 milestone Dec 15, 2017
@LavrovArtem LavrovArtem self-assigned this Dec 15, 2017
@LavrovArtem LavrovArtem mentioned this issue Dec 15, 2017
Closed
@Farfurix Farfurix self-assigned this Feb 22, 2018
@Farfurix
Copy link
Contributor

I'm working on this

@Farfurix Farfurix mentioned this issue Mar 7, 2018
Merged
AndreyBelym pushed a commit to AndreyBelym/testcafe-hammerhead that referenced this issue Feb 28, 2019
@Farfurix @AndreyBelym
fix 'Page without slash after location's host reloads if we set only …
db6500b 
…hash' (close DevExpress#1426) (DevExpress#1514)

* i1426

* i1426

* Requested changes: rename ensureHostEndedTrailingSlash() to ensureOriginTrailingSlash(); rewrite urlUtils.getProxyUrl(ensureOriginTrailingSlash(url))

* Fix server test (Should procees "x-frame-options" header (DevExpressGH-1017)), export TRAILING_SLASH_RE needed in this test, fix client location test

* Fix navigateTo(): leading slashes case

* Fix ensureOriginTrailingSlash(), code cleanup

* Add ensureOriginTrailingSlash test

* Fix location header, fix ensureOriginTrailingSlash test, add location header test, add location href test (wrapper)

* Add tests

* Requested changes

* Requested changes

* Requested changes
@lock
Copy link

lock bot commented Mar 28, 2019

This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests. We recommend you ask TestCafe API, usage and configuration inquiries on StackOverflow.

@lock lock bot added the STATE: Auto-locked Issues that were automatically locked by the Lock bot label Mar 28, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Mar 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@LavrovArtem LavrovArtem

@Farfurix Farfurix

Labels
AREA: client STATE: Auto-locked Issues that were automatically locked by the Lock bot SYSTEM: URL processing TYPE: bug
Projects
None yet
Milestone
Sprint #10
Development

No branches or pull requests
2 participants
@LavrovArtem @Farfurix