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

UTF7 repack corrected #34

Closed
wants to merge 1 commit into from
Closed

UTF7 repack corrected #34

wants to merge 1 commit into from

Conversation

alechner
Copy link

@alechner alechner commented Apr 24, 2018

function _repack_utf7_callback is converting incorrectly non utf7 strings.
example: Chassis+FanTray10G-VSS is removed from original html (text)

when added a test to detect if base64_encode(base64_decode($str))===$str
PHP was thrown an error: return must be string, returned array


This change is Reviewable

function _repack_utf7_callback is converting incorrectly non utf7 strings.
example:  Chassis+FanTray10G-VSS  is removed from original html (text)

when added a test to detect if base64_encode(base64_decode($str))===$str
PHP was thrown an error: return must be string, returned array
@coveralls
Copy link

coveralls commented Apr 24, 2018

Coverage Status

Coverage decreased (-1.8%) to 97.122% when pulling 71573ef on alechner:patch-1 into d8da4c9 on voku:master.

voku added a commit that referenced this pull request Apr 26, 2018
voku added a commit that referenced this pull request Apr 26, 2018
-> #34

# Conflicts:
#	src/voku/helper/AntiXSS.php
voku added a commit that referenced this pull request Apr 26, 2018
-> #34

# Conflicts:
#	src/voku/helper/AntiXSS.php
@voku
Copy link
Owner

voku commented Apr 26, 2018

Thank you for the fix. There was a issue with the string comparison, so I added the fix in a new commit + a test case for your example.

@voku voku closed this Apr 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants