mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #43356
[Bug 1707076] [NEW] Skin title not escaped in page settings form
Public bug reported:
When testing https://bugs.launchpad.net/mahara/+bug/1706536 I noticed
there was a problem on the page settings form where skin title was not
being escaped.
To test:
1) Set up a skin with the title:
It's all <script>alert(1);</script>good!
2a) If the patch for bug 1706536 is in play it should show the title as inputed but not execute the js
2b) If the patch for bug 1706536 is not present it should show the title with special characters escaped but not execute the js
3) Go to pages and collections and edit a page
4) Click on settings
You get an alert box with '1' in it
The title for the skin needs to be escaped/made safe
** Affects: mahara
Importance: High
Assignee: Cecilia Vela Gurovic (ceciliavg)
Status: Confirmed
** Affects: mahara/17.10
Importance: High
Assignee: Cecilia Vela Gurovic (ceciliavg)
Status: Confirmed
** Also affects: mahara/17.10
Importance: Undecided
Status: New
** Changed in: mahara/17.10
Importance: Undecided => High
** Changed in: mahara/17.10
Assignee: (unassigned) => Cecilia Vela Gurovic (ceciliavg)
** Changed in: mahara/17.10
Milestone: None => 17.10.0
** Changed in: mahara/17.10
Status: New => Confirmed
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1707076
Title:
Skin title not escaped in page settings form
Status in Mahara:
Confirmed
Status in Mahara 17.10 series:
Confirmed
Bug description:
When testing https://bugs.launchpad.net/mahara/+bug/1706536 I noticed
there was a problem on the page settings form where skin title was not
being escaped.
To test:
1) Set up a skin with the title:
It's all <script>alert(1);</script>good!
2a) If the patch for bug 1706536 is in play it should show the title as inputed but not execute the js
2b) If the patch for bug 1706536 is not present it should show the title with special characters escaped but not execute the js
3) Go to pages and collections and edit a page
4) Click on settings
You get an alert box with '1' in it
The title for the skin needs to be escaped/made safe
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1707076/+subscriptions
Follow ups
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Niranjan, 2018-02-22
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-10-30
-
[Bug 1707076] A change has been merged
From: Mahara Bot, 2017-10-06
-
[Bug 1707076] A patch has been submitted for review
From: Mahara Bot, 2017-10-06
-
[Bug 1707076] A change has been merged
From: Mahara Bot, 2017-10-06
-
[Bug 1707076] A patch has been submitted for review
From: Mahara Bot, 2017-10-04
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-09-11
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Cecilia Vela Gurovic, 2017-09-10
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-09-10
-
[Bug 1707076] A change has been merged
From: Mahara Bot, 2017-09-04
-
[Bug 1707076] A change has been merged
From: Mahara Bot, 2017-09-04
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-09-04
-
[Bug 1707076] A change has been merged
From: Mahara Bot, 2017-09-04
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-09-04
-
[Bug 1707076] A patch has been submitted for review
From: Mahara Bot, 2017-09-04
-
[Bug 1707076] A patch has been submitted for review
From: Mahara Bot, 2017-09-04
-
[Bug 1707076] A patch has been submitted for review
From: Mahara Bot, 2017-09-04
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-09-04
-
[Bug 1707076] A change has been merged
From: Mahara Bot, 2017-08-14
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Robert Lyon, 2017-08-14
-
[Bug 1707076] A patch has been submitted for review
From: Mahara Bot, 2017-07-31
-
[Bug 1707076] Re: Skin title not escaped in page settings form
From: Cecilia Vela Gurovic, 2017-07-31