Comments (6)
You bring up a good point. Should we htmlentities()
the entire output? I don't think I've ever krumo'd HTML strings before.
Should be a simple fix if we can figure out what ELSE might need escaping.
from krumo.
Might not be a bad idea...(htmlentities)
On May 4, 2015, at 10:32 AM, Scott Baker [email protected] wrote:
You bring up a good point. Should we htmlentities() the entire output? I
don't think I've ever krumo'd HTML strings before.
—
Reply to this email directly or view it on GitHub
https://github.com/oodle/krumo/issues/26#issuecomment-98787797.
from krumo.
I'm really surprised we haven't run in to this before. I'll have to do some investigation. Should be an easy fix though.
from krumo.
Sorry it too me so long to look at this. I'm not seeing the problem you're seeing. If I look at the code we're already escaping string output:
https://github.com/oodle/krumo/blob/master/class.krumo.php#L1452
Are you sure you're running the latest version?
from krumo.
@dmyers2004 any update on this issue?
from krumo.
Closing this issue because it's been more than 2 years. Will to re-open if it's still an issue.
from krumo.
Related Issues (20)
- krumo provide more info on a array-casted object than on the object itself HOT 4
- Feature request (and patch) : shift+click to expand all HOT 8
- Javascript bug in version 0.5.6 HOT 4
- Non-static method called statically HOT 14
- Typo in README.md "Usage" section example HOT 1
- Don't truncate after 50 characters HOT 5
- Undefined index: class:: HOT 7
- Update Packagist? HOT 3
- How to change skin? HOT 17
- krumo not showing private and protected properties HOT 2
- Is it possible to 'queue' so they can be displayed in bulk at once HOT 2
- Dropping VERSION file in favor of a class constant HOT 1
- Using a separate method for returning Krumo dumps instead using KRUMO_RETURN as argument HOT 1
- Drop the INSTALL file HOT 1
- Put composer.lock in .gitignore and drop it HOT 3
- Drop "@static" and "@access public" HOT 2
- Drop KRUMO_DIR, replace with __DIR__ HOT 1
- Change how configuration is operated HOT 2
- Krumo doesn't cleanly support PHP 8.1 HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from krumo.