Comments (8)
slandelle
commented on August 16, 2024
1
This is a bug with Jodd's IE conditional comments support:
<html><head>
<!--[if lte IE 9]><meta http-equiv="refresh" content="1;url=/2.218.0/static/unsupp.html" /><![endif]-->
<!--[if gt IE 9><!--><script>alert("Hello!");</script><!--<![endif]-->
</head>
</html>The first line works fine, the second one crashes with IOOBE.
A possible workaround is to disable IE conditional comments support in Lagarto config.
from jodd.
igr
commented on August 16, 2024
1
I checked the code, you didnt make mistake @slandelle
from jodd.
slandelle
commented on August 16, 2024
1
Regular comment IMHO
from jodd.
igr
commented on August 16, 2024
1
Releasing new version @slandelle atm :)
from jodd.
igr
commented on August 16, 2024
Thanx @slandelle for fast detection! It seems that the second line has the syntax error for Conditional Comments, i.e. its missing the ]. I guess that in this case this is not a conditional comment, but regular one. Jodd should 'roll back' the conditional condition parsing and go with comment (just need to verify what happens in browser... internet explorer, anyone?:))).
from jodd.
slandelle
commented on August 16, 2024
Let me check if I didn’t introduce a typo when simplifying cnn.com HTML content.
from jodd.
igr
commented on August 16, 2024
if I assume that [if gt IE 9> is a regular comment, should Jodd emit conditional comment on endif or regular comment (as the starting conditional comment does not exist)?
from jodd.
slandelle
commented on August 16, 2024
Thanks mate!
from jodd.
Related Issues (20)
- JoyInit beans should be invoked after the Joy printouts
- optional return types for jodd-json HOT 3
- jodd-http set body encoding HOT 2
- Why LFUCache doesnot initialize with a loadFactor=1.0f of a Hashmap? HOT 14
- java.lang.IndexOutOfBoundsException HOT 17
- When I send one email with attachment, the attachment name is gibberish in Chinese. HOT 2
- envelopeOnly() results in lots of fetching compared to folder.search HOT 2
- java.lang.IndexOutOfBoundsException for 27 pages - tested 862_674 pages. HOT 42
- jodd-petite singleton scope bean is thread safe?? HOT 1
- petite problem: when invoke addBean will throw MalformedParametersException HOT 4
- Can you integrate Swagger Doc like Spring frame? HOT 1
- Duplicate class HOT 22
- fix jodd-db document HOT 1
- InterfaceImplementation.java ClassCastException when interface return primitive type but not number HOT 1
- [How to] register custom action annotation? HOT 1
- Required OSGi headers missing HOT 3
- IDEA: Provide PathMacros in util module HOT 1
- LRUCache has Concurrency problem HOT 5
- tag missing problem
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 jodd.