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

Revert "Strip null bytes from strings and filter conditions." #2600

Closed
wants to merge 1 commit into from

Conversation

fballiano
Copy link
Contributor

Reverts #1430 due to #1430 (comment)

@github-actions github-actions bot added Component: Core Relates to Mage_Core Component: lib/Varien Relates to lib/Varien Component: lib/* Relates to lib/* labels Sep 14, 2022
@github-actions

This comment has been minimized.

1 similar comment
@github-actions
Copy link
Contributor

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
7 runs  ±0  5 ✔️ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 9a58f76. ± Comparison against base commit c27098e.

@elidrissidev
Copy link
Member

Just faced this issue today too.

@sreichel
Copy link
Contributor

sreichel commented Sep 14, 2022

Has someone time to test this: #1430 (comment)

If not lets merge an reapply @colinmollenhour fixed PR again.

@sreichel sreichel mentioned this pull request Sep 14, 2022
3 tasks
@fballiano fballiano closed this Sep 14, 2022
@fballiano fballiano deleted the revert-1430-strip-null-bytes branch September 14, 2022 22:17
@ADDISON74
Copy link
Contributor

ADDISON74 commented Sep 14, 2022

I think it was better to merge this revert in the Git history. Later a new PR could be created to come up with the fix to the initial PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Core Relates to Mage_Core Component: lib/Varien Relates to lib/Varien Component: lib/* Relates to lib/*
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants