Conversation
|
Looking into it. I want to understand what it was disabled. |
|
I attempted to uncomment both of those lines and that results in errors like this during test/mri/cgi tests: And the issue in #4678 still causes a similar error attempting to call a super |
|
ok, if it's too risky, feel free to postpone it |
|
Yeah I'll postpone it. There's something not right about how we're putting this particular class hierarchy together, or else something wrong with the way super searches that hierarchy. I'll prioritize this for 9.2.10 so we can get rid of the remaining hacks and workarounds. |
|
I realized a simple way we can integrate this: implement it as a separate ext that does not have to be prepended and supered in the ways MRI does. Looking into this quickly as an option. |
as commented in #5937
CGI.escapeHTMLdoes always use a pure ruby version which is slowerhttps://github.com/ruby/ruby/blob/master/ext/cgi/escape/escape.c#L408