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

Selenide proxy should allow mocking server response AND specify content type #2705

Closed
asolntsev opened this issue Apr 3, 2024 · 0 comments · Fixed by #2706
Closed

Selenide proxy should allow mocking server response AND specify content type #2705

asolntsev opened this issue Apr 3, 2024 · 0 comments · Fixed by #2706
Assignees
Milestone

Comments

@asolntsev
Copy link
Member

The problem

Currently it's possible to mock server response content, but not content type.
The mocked response doesn't have Content-Type header, and some applications might not work without it.

Details

See detailed description at #1978 (comment)

Tell us about your environment

  • Selenide Version: 7.2.3
@asolntsev asolntsev self-assigned this Apr 3, 2024
@asolntsev asolntsev added this to the 7.2.4 milestone Apr 3, 2024
asolntsev added a commit that referenced this issue Apr 4, 2024
To allow user set content type, we leverage Selenium own class "org.openqa.selenium.remote.http.HttpResponse" that was already used by Selenium in CDP-specific code. It allows to set  status+headers+content.
asolntsev added a commit that referenced this issue Apr 4, 2024
To allow user set content type, we leverage Selenium own class "org.openqa.selenium.remote.http.HttpResponse" that was already used by Selenium in CDP-specific code. It allows to set  status+headers+content.
asolntsev added a commit that referenced this issue Apr 4, 2024
To allow user set content type, we leverage Selenium own class "org.openqa.selenium.remote.http.HttpResponse" that was already used by Selenium in CDP-specific code. It allows to set  status+headers+content.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant