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

remove_invisible_characters removing legitimate paste content. #28

Closed
fmichea opened this issue Sep 21, 2012 · 4 comments
Closed

remove_invisible_characters removing legitimate paste content. #28

fmichea opened this issue Sep 21, 2012 · 4 comments
Milestone

Comments

@fmichea
Copy link

@fmichea fmichea commented Sep 21, 2012

Hi guys,

I installed and started using your pastebin last week, and today we noticed that some legitimate content (example bellow) was removed. I looked in the code and think it's due to remove_invisible_characters[1] removing HTML special charaters pasted in hexadecimal (I think).

Is this a feature? Could it be something wrong in my configuration? Is there a work arround?

Example (C code):

int main(void)
{
    // When pasted, only "s" remains as the format string.
    printf("%12s", "I am a string");
    return 0;
}

I am sorry if it is not the right place for this. Thank you in advance.

[1] https://github.com/claudehohl/Stikked/blob/master/htdocs/system/core/Common.php#L514

@claudehohl
Copy link
Owner

@claudehohl claudehohl commented Sep 22, 2012

Indeed, a nasty "bug". Thanks to the fact that codeigniter allows us to replace core libs within our application (http://codeigniter.com/user_guide/general/core_classes.html), I could comment that replacement out. Makes no sense for codesharing pastebins.

Here's your new paste: http://paste.scratchbook.ch/view/ef230571

@claudehohl claudehohl closed this Sep 22, 2012
@claudehohl
Copy link
Owner

@claudehohl claudehohl commented Sep 22, 2012

And of course, the corresponding commit: ef1d6ed

Just put the Input.php into application/core/

skorotkiewicz pushed a commit to skorotkiewicz/Stikked that referenced this issue Oct 25, 2014
@claudehohl
Copy link
Owner

@claudehohl claudehohl commented Sep 1, 2015

test it with CI 3.0

@claudehohl claudehohl reopened this Sep 1, 2015
@claudehohl claudehohl added this to the 0.9.1 milestone Sep 1, 2015
@claudehohl
Copy link
Owner

@claudehohl claudehohl commented Sep 2, 2015

tested with ci 3, works.

@claudehohl claudehohl closed this Sep 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.