Project

General

Profile

Bug #2159

problem when inserting a google map in the forum

Added by Jose Manuel Abuin Mosquera over 9 years ago. Updated over 7 years ago.

Status:
Bug resolved
Priority:
Low
Assignee:
Julio Montoya
Category:
-
Target version:
1.9 RC1
Start date:
14/10/2010
Due date:
% Done:

100%

Estimated time:
Spent time:
0.70 h
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Chamilo 1.8.7.1 on Debian Lenny

Hi guys,

we just noticed a problem when inserting a google map in the forum of a course. No matter if you do it with the "google map" button or by inserting the link provided by google maps in "html view" mode, the map does not appear. If you use the google map button it request you insert a google maps api...but that does not have much sense.

Anyone knows how to solve this?

Thanks in advance!

Regards

Associated revisions

Revision a68fb555 (diff)
Added by Julio Montoya over 7 years ago

Adding Db change for the iframe inclusion see #2159

Revision bc3d66fd (diff)
Added by Julio Montoya over 7 years ago

Adding enable_iframe_inclusion setting + htmlpurifier filter see #2159

Revision 7c87b553 (diff)
Added by Yannick Warnier over 7 years ago

Added documentation to iframe filter for HTMLPurifier - refs #2159

History

#1

Updated by Yannick Warnier over 7 years ago

  • Target version set to 1.9 Beta
#2

Updated by Julio Montoya over 7 years ago

  • Status changed from New to Needs more info
  • Priority changed from Normal to Low

A new filter should be added to HTMLpurifier in order to allow iframes but this means adding security risks

http://htmlpurifier.org/phorum/read.php?3,4646,5029

#3

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs more info to Assigned
  • Assignee set to Julio Montoya

Then we should add a security parameter:
$GlobalSettingAllowIFramesTitle = "Allow iframes in HTML Editor";
$GlobalSettingAllowIFramesComment = "Allowing arbitrary iframes in the HTML Editor will enhance the edition capabilities of the users, but it can represent a security risk. Please make sure you can rely on your users (i.e. you know who they are) before enabling this feature.";

Variable name: wysiwyg_enable_iframe_inclusion (should be configurable by sub-url)

#4

Updated by Julio Montoya over 7 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 0 to 80

code added it only supports vimeo, youtube and google maps

#5

Updated by Yannick Warnier over 7 years ago

  • Target version changed from 1.9 Beta to 1.9 RC1
#6

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs more info to Bug resolved
  • Assignee set to Julio Montoya
  • % Done changed from 80 to 100

OK at code level.

Also available in: Atom PDF