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

require-post-message-target-origin should ignore some cases #1494

Closed
2000yeshu opened this issue Aug 19, 2021 · 3 comments
Closed

require-post-message-target-origin should ignore some cases #1494

2000yeshu opened this issue Aug 19, 2021 · 3 comments
Labels

Comments

@2000yeshu
Copy link

2000yeshu commented Aug 19, 2021

This rule throws an error when using the postMessage method window.Worker but it shouldn't. For example,

this.worker = new Worker('/speedtest_worker.js');
this.worker.postMessage('some message');

It should ignore this case because there is no target origin parameter in the postmessage method of worker.

@2000yeshu 2000yeshu added the bug label Aug 19, 2021
@2000yeshu
Copy link
Author

I would love to fix this issue. You can assign this issue to me.

@sindresorhus
Copy link
Owner

Please include a code full example of the cases it should ignore.

@fisker
Copy link
Collaborator

fisker commented Aug 20, 2021

Duplicate of #1396

@fisker fisker marked this as a duplicate of #1396 Aug 20, 2021
@fisker fisker closed this as completed Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants