Skip to content

fix(test): move xss test from 942180 to 941210 #4012

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.

Sign up for GitHub

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

Jump to bottom
Merged
theseion merged 4 commits into from
Jul 12, 2025
Merged

fix(test): move xss test from 942180 to 941210 #4012

theseion merged 4 commits into from
Jul 12, 2025

Conversation

@fzipi
Copy link
Member

@fzipi fzipi commented Feb 24, 2025 •
edited
Loading

what

  • move test wrongfully matching 942180 to 941210

why

  • test is targeted to match xss not sqli

references

@fzipi fzipi added the release:ignore Ignore for changelog release label Feb 24, 2025
@fzipi fzipi requested a review from theseion February 24, 2025 00:44
theseion
theseion requested changes Feb 24, 2025
View reviewed changes
@fzipi fzipi enabled auto-merge February 24, 2025 10:40
@fzipi fzipi requested a review from theseion February 24, 2025 10:40
@theseion
Copy link
Contributor

theseion commented Feb 24, 2025

Looks good but tests are failing.

@fzipi fzipi force-pushed the tests/942180-fix-test branch 2 times, most recently from acb8e1b to c27f145 Compare February 24, 2025 16:37
@fzipi fzipi mentioned this pull request Mar 3, 2025
Closed
@fzipi fzipi mentioned this pull request Apr 7, 2025
Closed
@github-actions github-actions bot added the Stale label Apr 26, 2025
@Xhoenix Xhoenix removed the Stale label Apr 27, 2025
@fzipi fzipi mentioned this pull request May 5, 2025
Closed
@github-actions github-actions bot added the Stale label May 28, 2025
@fzipi fzipi mentioned this pull request Jun 2, 2025
Closed
@github-actions github-actions bot closed this Jun 11, 2025
auto-merge was automatically disabled June 11, 2025 00:32

Pull request was closed

@theseion theseion reopened this Jun 11, 2025
@github-actions github-actions bot removed the Stale label Jun 12, 2025
@fzipi fzipi mentioned this pull request Jul 7, 2025
Closed
@fzipi fzipi force-pushed the tests/942180-fix-test branch from 6325af9 to 122afe9 Compare July 11, 2025 21:13
@fzipi
fix: move test to 941210
79f8fc1 
Signed-off-by: Felipe Zipitria <[email protected]>
@fzipi fzipi force-pushed the tests/942180-fix-test branch from 122afe9 to 79f8fc1 Compare July 11, 2025 21:25
@fzipi fzipi requested a review from theseion July 11, 2025 21:44
@fzipi fzipi changed the title fix(test): move xss test from 942180 to 941330 fix(test): move xss test from 942180 to 941210 Jul 11, 2025
@theseion theseion added this pull request to the merge queue Jul 12, 2025
Merged via the queue into main with commit 824f264 Jul 12, 2025
10 checks passed
@theseion theseion deleted the tests/942180-fix-test branch July 12, 2025 07:06
@fzipi fzipi mentioned this pull request Aug 4, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@theseion theseion theseion approved these changes

Assignees

No one assigned

Labels

release:ignore Ignore for changelog release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fzipi @theseion @Xhoenix