Infinite Loop because of HTML

32 Posts

gboyd1978 Reply #1, 13 years, 10 months ago

Hey guys,

I am having issues with some template when I past the following html into the content area I try to save and get an never ending saving message. Can someone else please try and save the following html into an area and see if they get the same thing. I know the html is messy but its getting pasted in from Campaign Monitor.

Any help would be appreciated this is driving me crazy.

I am currently on version 2.2.6

<p><span> </span></p>
<table class="w640" width="640" border="0" cellspacing="0" cellpadding="0">
<tbody>
<tr style="border-collapse: collapse;">
<td class="w30" style="border-collapse: collapse;" width="30"> </td>
<td class="w580" style="border-collapse: collapse;" width="580">
<table class="w580" width="580" border="0" cellspacing="0" cellpadding="0">
<tbody>
<tr style="border-collapse: collapse;">
<td class="w580" style="border-collapse: collapse;" width="580">
<p class="article-title underline" style="font-size: 22px; color: #752828; font-weight: bold; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif; line-height: 20px; width: 100%; border-bottom-width: 1px; border-bottom-style: solid; border-bottom-color: #CCC; margin-top: 10px; margin-bottom: 20px; padding-bottom: 5px;" align="left">Bandol Within Reach</p>
<div class="article-content" style="font-size: 14px; line-height: 18px; color: #333; margin-top: 0px; margin-bottom: 10px; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif;" align="left">
<p style="color: #333; margin-bottom: 15px;">I have a love affair going on with the wines of Bandol. I don’t discriminate, I love all of the Kermit Lynch Bandols and get giddy when they arrive. My Bandol collection might rival some women’s shoe collections, if I had to take stock (but won’t, lest Jeff get hip to this).</p>
<p style="color: #333; margin-bottom: 15px;">Of all the Bandols Kermit represents, Terrebrune is perhaps the most polished and the first to mature, but that's because it is the least tannic, and that by no means indicates a wine that can't age. To the contrary, it may well be the one that ages the longest because of its balance.</p>
<p style="color: #333; margin-bottom: 15px;">My love of Bandol is shared by Bruce Neyers, National Sales Manger for Kermit Lynch, who has inspired my thirst with his commentary on Terrebrune:</p>
<p style="color: #333; margin-bottom: 15px;">“I love our visits to Domaine Terrebrune. There is obviously something about the wines of Bandol that agrees with me as I find myself opening a bottle of one every week, or more. As fond as I am for the Bandol from Domaine Tempier, however, we get very little of it, so I always feel a bit guilty taking any home. That means less for our clients. Domaine Terrebrune got me over that problem, though, and faster than a therapist.</p>
<p style="color: #333; margin-bottom: 15px;">Their vines are grown in the rocky, dark brown (Terra Brune) soil of Ollioules, several miles closer to the Mediterranean than Tempier, so the climate is slightly moderated by the sea breeze. The wines are similar in style, however, and easily mistaken for one another. Much of that has to do with the fact that the late Lucien Peyraud, founder of Tempier, actually oversaw the development of Terrebrune's vineyards.</p>
<p style="color: #333; margin-bottom: 15px;">Reynald Delille is the guiding light behind Domaine Terrebrune and he offers the latest wines from this unique terroir. A small, attractive restaurant is part of the domaine as well so they maintain an extensive collection of older bottlings for their dining customers to enjoy.”</p>
</div>
</td>
</tr>
<tr style="border-collapse: collapse;">
<td class="w580" style="border-collapse: collapse;" width="580" height="10"> </td>
</tr>
</tbody>
</table>
<table class="w580" width="580" border="0" cellspacing="0" cellpadding="0">
<tbody>
<tr style="border-collapse: collapse;">
<td class="w580" style="border-collapse: collapse;" width="580" height="10px"><hr style="margin-bottom: 10px; margin-top: 10px; border-width: 0; color: #ccc; background-color: #ccc; height: 1px; width: 100%; text-align: left; padding: 0px;" /></td>
</tr>
</tbody>
</table>
<table class="w580" border="0" cellspacing="0" cellpadding="0">
<tbody>
<tr style="border-collapse: collapse;">
<td class="w580" style="border-collapse: collapse;" width="580">
<p class="article-title" style="margin-top: 0; margin-bottom: 0px; margin-right: 0; margin-left: 0; padding-top: 0; padding-bottom: 0px; padding-right: 0; padding-left: 0; font-size: 22px; line-height: 24px; color: #752828; font-weight: bold; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif;" align="left">2007 Bandol Rouge, Certified Organic</p>
<p class="article-subtitle" style="margin-top: 0; margin-bottom: 10px; margin-right: 0; margin-left: 0; padding-top: 0; padding-bottom: 0px; padding-right: 0; padding-left: 0; font-size: 18px; font-weight: bold; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif; color: #752828;" align="left">Domaine de Terrebrune</p>
<table border="0" cellspacing="0" cellpadding="0" align="right">
<tbody>
<tr style="border-collapse: collapse;">
<td class="w30" style="border-collapse: collapse;" width="15"> </td>
<td style="border-collapse: collapse;"><img class="w240" style="height: auto; line-height: 100%; outline-style: none; text-decoration: none; display: block;" src="http://i2.createsend1.com/ei/r/59/598/AD7/csimport/Terrebrune.170740.jpg" alt="" width="160" height="160" border="0" /></td>
</tr>
<tr style="border-collapse: collapse;">
<td class="w30" style="border-collapse: collapse;" width="15" height="5"> </td>
<td style="border-collapse: collapse;"> </td>
</tr>
</tbody>
</table>
<div class="article-content" style="font-size: 14px; line-height: 18px; color: #333; margin-top: 0px; margin-bottom: 10px; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif;" align="left">
<p style="color: #333; margin-bottom: 15px;">Not only am I inspired to open a bottle of Bandol this evening, but inspired to incorporate a visit to Domaine Terrebrune on my next trip to France!</p>
<p style="color: #333; margin-bottom: 15px;"><span style="color: #752828;"><strong>85% Mourvèdre<br style="line-height: 100%;" /> 10% Grenache<br style="line-height: 100%;" /> 5% Cinsault</strong></span></p>
<p style="color: #333; margin-bottom: 15px;">Important in Ollioules is to be able to see the Mediterranean as it serves to cool the otherwise harsh climate, and the evaporative effect leaves a touch of ‘saltiness’ on the grapes that becomes a flavour component in the wine.</p>
<p style="color: #333; margin-bottom: 15px;"><strong>The Colour:</strong> immediately engaging, as it has elements of purple, ruby, and a touch of emerald as well.</p>
<p style="color: #333; margin-bottom: 15px;"><strong>The Nose: </strong>animal and pit fruit with traces of peach, cassis, and blueberry, which as it unfolds shows signs of that delicious combination of licorice and garrigue.</p>
<p style="color: #333; margin-bottom: 15px;"><strong>The Taste:</strong> sweet flavours, youthful but soft, with a bare trace of salinity.</p>
</div>
<p style="font-family: 'Lucida Sans Unicode', 'Lucida Grande' , sans-serif; color: #666; margin-top: 10px; margin-bottom: 14px; margin-right: 0; margin-left: 0; font-size: 16px; font-weight: bold; padding: 0;" align="left">6x750ml                         Sold Out</p>
<p style="font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif; font-size: 22px; font-weight: bold; color: #752828; margin-top: 0; margin-bottom: 4px; margin-right: 0; margin-left: 0; padding-top: 0; padding-bottom: 4px; padding-right: 0; padding-left: 0;" align="left">$39.97/bt</p>
<p style="font-family: 'Lucida Sans Unicode', 'Lucida Grande' , sans-serif; color: #333333; margin-top: 10px; margin-bottom: 14px; margin-right: 0; margin-left: 0; font-size: 14px; padding: 0;" align="left"><span style="font-size: 16px; font-weight: bold; color: #333333;">If Tempier is out of reach or hidden in your cellar, Terrebrune is your answer. How about sharing in our Bandol Wine Joy and picking up some Terrebrune ?</span></p>
</td>
</tr>
</tbody>
</table>
<table class="w580" width="580" border="0" cellspacing="0" cellpadding="0">
<tbody>
<tr style="border-collapse: collapse;">
<td class="w580" style="border-collapse: collapse;" width="580">
<p class="article-title-2" style="font-size: 18px; line-height: 20px; color: #752828; font-weight: bold; margin-top: 0px; margin-bottom: 18px; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif;" align="left"> </p>
<table border="0" cellspacing="0" cellpadding="0" align="left">
<tbody>
<tr style="border-collapse: collapse;">
<td style="border-collapse: collapse;"> </td>
<td class="w30" style="border-collapse: collapse;" width="15"> </td>
</tr>
<tr style="border-collapse: collapse;">
<td style="border-collapse: collapse;"> </td>
<td class="w30" style="border-collapse: collapse;" width="15" height="5"> </td>
</tr>
</tbody>
</table>
<div class="article-content" style="font-size: 14px; line-height: 18px; color: #333; margin-top: 0px; margin-bottom: 10px; font-family: 'Lucida Sans Unicode', 'Lucida Grande', sans-serif;" align="left">
<p style="margin-top: 10px; color: #752828; font-size: 22px; font-weight: bold; margin-bottom: 15px;">D</p>
</div>
</td>
</tr>
<tr style="border-collapse: collapse;">
<td class="w580" style="border-collapse: collapse;" width="580" height="5"> </td>
</tr>
</tbody>
</table>
</td>
<td class="w30" style="border-collapse: collapse;" width="30"> </td>
</tr>
</tbody>
</table>

MODX Staff
95 Posts

waldus68 Reply #2, 13 years, 10 months ago

I have run into this issue many times before and it always ended up being ModSecurity preventing it from saving. You may need to get in touch with your host to see if that is the case and if they can help fix it.

☆ A M B ☆
279 Posts

bertoost Reply #3, 13 years, 10 months ago

I have test your HTML code in my 2.2.6 test installation of MODX and it saves correctly.. So think you should use your browser inspector (F12 I guess?) to see what the ajax processor tells you, maybe you get an error somehow, so investigate ;-)

MODX Ambassador (NL) & Professional MODX developer
Follow me on Twitter | Visit my page on Facebook | View my code on Github | View my script posts
MODX e-commerce solution SimpleCart

32 Posts

gboyd1978 Reply #4, 13 years, 10 months ago

Quote from: waldus68 at Dec 14, 2012, 09:57 AM

I have run into this issue many times before and it always ended up being ModSecurity preventing it from saving. You may need to get in touch with your host to see if that is the case and if they can help fix it.

I have attached my error within and I am getting a 403 error what exactly would I ask them because my hosting blames it on modx and does not help me with development

MODX Staff
95 Posts

waldus68 Reply #5, 13 years, 10 months ago

I can't say for certain that is what is causing the issue, just noting that I have had a simliar issues in past and that turned out to be the culprit. In my scenario, I would ask the host if they were running ModSecurity and if it could be blocking any of the manager files. I would also provide my IP address to them so they could find it in the logs. With that information they would respond that they whitelist the rule for the affected files and the issue would be fixed.

This could be a different issue but thought it might be a helpful thing to check on since the symptoms sound very simliar. Hopefully that helps you track down the issue.

32 Posts

gboyd1978 Reply #6, 13 years, 10 months ago

Hi I appreciate your response. I don't understand how this could start happening because at one time I could save these files. If I remove the above copy from the page it saves fine... its a very hard thing to debug and my client is getting very upset that he cant save pages.

Again I appreciate your help .. Happy Holidays

☆ A M B ☆
24,524 Posts

sottwell Reply #7, 13 years, 10 months ago

Hosts often make updates and changes to PHP versions, mod_security or suhosin settings, and other things that break sites with no warning. There is probably some word or HTML structure that has been added to a security setting. For example, there are some nasty site injection exploits that use iframes, so some are preventing iframes from being allowed in the POST.

Any decent hosting company will whitelist the /manager/index.php file from their security settings if you ask them to.

Studying MODX in the desert - http://sottwell.com
Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
Join the Slack Community - http://modx.org

32 Posts

gboyd1978 Reply #8, 13 years, 10 months ago

Quote from: sottwell at Dec 14, 2012, 02:57 PM

Hosts often make updates and changes to PHP versions, mod_security or suhosin settings, and other things that break sites with no warning. There is probably some word or HTML structure that has been added to a security setting. For example, there are some nasty site injection exploits that use iframes, so some are preventing iframes from being allowed in the POST.

Any decent hosting company will whitelist the /manager/index.php file from their security settings if you ask them to.

Thanks for the Reply would that be the only file I should actually whitelist... my hosting is asking for a list.

32 Posts

gboyd1978 Reply #9, 13 years, 10 months ago

I actually have tracked down the line of code which is causing the infinite loop.

<hr style="margin-bottom: 10px; margin-top: -10px; border-width: 0; color: #ccc; background-color: #ccc; height: 1px; width: 100%; text-align: left; padding: 0px;" />

Totally random why that should cause Modx to goo into a repeating loop.

☆ A M B ☆
24,524 Posts

sottwell Reply #10, 13 years, 10 months ago

Interesting. A look at the logs would tell you exactly what word or phrase is causing the security check to fail.

Anyway, the /manager/index.php file is probably the only one you need to have whitelisted, unless you will be doing front-end resource editing such as NewsPublisher. There are really only two .php files that are run directly, the root index.php for front-end pages, and /manager/index.php for Manager work. All the rest of MODx's files are included, not run independently.

Studying MODX in the desert - http://sottwell.com
Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
Join the Slack Community - http://modx.org