8000 fix: update xss detection with onwebkitplaybacktargetavailabilitychanged event by fzipi · Pull Request #3822 · coreruleset/coreruleset · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: update xss detection with onwebkitplaybacktargetavailabilitychanged event #3822

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

Merged
merged 4 commits into from
Sep 16, 2024

Conversation

fzipi
Copy link
Member
@fzipi fzipi commented Sep 13, 2024

Followup for PR #3734, originally from @Osb0rn3

@fzipi fzipi changed the title Followup pr3734 osb0rn3 fix: update xss detection with onwebkitplaybacktargetavailabilitychanged event Sep 13, 2024
@fzipi fzipi force-pushed the followup-pr3734-Osb0rn3 branch 3 times, most recently from 9bbe660 to 30d78c7 Compare September 13, 2024 14:48
@theseion theseion force-pushed the followup-pr3734-Osb0rn3 branch 4 times, most recently from 0e471a7 to cd2c42b Compare September 14, 2024 07:48
@fzipi fzipi force-pushed the followup-pr3734-Osb0rn3 branch from 3a89f8d to 18e4f09 Compare September 16, 2024 15:05
Osb0rn3 and others added 4 commits September 16, 2024 12:36
- Updated to handle 50 characters on-prefixed HTML attributes.
- Added new event handler for onwebkitplaybacktargetavailabilitychanged to Role ID `941160`.
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
@fzipi fzipi force-pushed the followup-pr3734-Osb0rn3 branch from ac2ea25 to 89e1879 Compare September 16, 2024 15:36
Copy link
Member
@dune73 dune73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but there are changes lint.yaml too. Is this correct?

@fzipi
Copy link
Member Author
fzipi commented Sep 16, 2024

Looks good, but there are changes lint.yaml too. Is this correct?

Yes, this is Max's fix for the linting issue.

@fzipi fzipi requested a review from dune73 September 16, 2024 15:57
@dune73 dune73 added this pull request to the merge queue Sep 16, 2024
Merged via the queue into main with commit e1806e0 Sep 16, 2024
6 checks passed
@dune73 dune73 deleted the followup-pr3734-Osb0rn3 branch September 16, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0