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

fix(plugin): prevent NPE on writable stream #102

Merged

Conversation

derevnjuk
Copy link
Member

@derevnjuk derevnjuk commented Nov 9, 2022

The ensureBrowserFlags function has been deprecated. Use the install function instead as follows:

setupNodeEvents(on) {
  install(on);
-
-  on('before:browser:launch', (browser = {}, launchOptions) => {
-    ensureBrowserFlags(browser, launchOptions);
-    return launchOptions;
-  });
}

In case of any issues please refer to cypress-io/cypress#5240

closes #100

Function has been deprecated. Use the `install` function instead as follows:
```diff
setupNodeEvents(on) {
  install(on);
-  // bind to the event we care about
-  on('before:browser:launch', (browser = {}, launchOptions) => {
-    ensureBrowserFlags(browser, launchOptions);
-    return launchOptions;
-  });
}
```
In case of any issues please refer to cypress-io/cypress#5240
@derevnjuk derevnjuk self-assigned this Nov 9, 2022
@derevnjuk derevnjuk added the Type: bug Something isn't working label Nov 9, 2022
@derevnjuk derevnjuk merged commit 68b0f3f into master Nov 9, 2022
@derevnjuk derevnjuk deleted the fix_#100/plugin_crashes_on_write_if_another_response_comes branch November 9, 2022 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plugin crashes on write if another response comes
1 participant