-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
cy.server whitelist behaves as blacklist #6642
Labels
type: breaking change
Requires a new major release version
Comments
yeeeaaah...I can see this. This would require a breaking change. |
jennifer-shehane
added
the
type: breaking change
Requires a new major release version
label
Mar 5, 2020
5 tasks
cypress-bot
bot
added
stage: work in progress
stage: needs review
The PR code is done & tested, needs review
and removed
stage: proposal 💡
No work has been done of this issue
stage: work in progress
labels
Jun 23, 2020
The code for this is done in cypress-io/cypress#7782, but has yet to be released. |
cypress-bot
bot
removed
the
stage: needs review
The PR code is done & tested, needs review
label
Jun 30, 2020
Released in 5.0.0. This comment thread has been locked. If you are still experiencing this issue after upgrading to Cypress v5.0.0, please open a new issue. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Current behavior:
Currently
whitelist
option forcy.server()
actually works as blacklist due to this linehttps://github.com/cypress-io/cypress/blob/master/packages/driver/src/cypress/server.coffee#L236
In other words:
will ignore any request
and
will catch all requests
Desired behavior:
Exactly opposite
should catch all requests
and
should ignore any request
Test code to reproduce
Versions
4.1.0
, also reproducible inmaster
win10, chrome
The text was updated successfully, but these errors were encountered: