All users were logged out of Bugzilla on October 13th, 2018

HTML from plugin's description should be stripped out

VERIFIED FIXED in 1.4

Status

--
major
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: stephend, Assigned: jsocol)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: aaq sumo_only, URL)

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
STR:

1. Load http://support-stage.mozilla.org/en-US/forum/1&openpost=1
2. Click on "Ask a question"
3. Choose "Problems with settings, bookmarks, passwords or other data stored by Firefox"
4. Click "Next step"
5. Type something, click "Next step"
6. Click on "Show details"

Actual Results:

*-Shockwave Flash 10.0 r32
*Runs Java applets using the latest installed versions of Java. For more information: <A HREF=http://javaplugin.sourceforge.net/>Java Embedding Plugin</A>. Run version test: <A HREF=http://browserspy.dk/java.php>Java Information</A>.
*Gecko default plugin
*The QuickTime Plugin allows you to view a wide variety of multimedia content in web pages. For more information, visit the <A HREF=http://www.apple.com/quicktime>QuickTime</A> Web site.
*3.0.40818.0
*The Flip4Mac WMV Plugin allows you to view Windows Media content using QuickTime.
*DivX Web Player version 1.4.1.4
*iPhoto6

Expected Results:

Above, we should strip out the raw HTML given to us by the plugin's description, namely "For more information: <A HREF=http://javaplugin.sourceforge.net/>Java Embedding Plugin</A>. Run version test: <A HREF=http://browserspy.dk/java.php>Java Information</A>." and turn it into a link (or just not show it here?)
(Assignee)

Updated

9 years ago
Assignee: nobody → james
Whiteboard: aaq
(Assignee)

Updated

9 years ago
Summary: HTML from plugin's description should be stripped out or marked up into a link → HTML from plugin's description should be stripped out
(Assignee)

Comment 1

9 years ago
Created attachment 402395 [details] [diff] [review]
patch, v1
Attachment #402395 - Flags: review?(laura)
(Assignee)

Comment 2

9 years ago
That patch also adds a header for the User Agent field.

Updated

9 years ago
Attachment #402395 - Flags: review?(laura) → review+
(Assignee)

Comment 3

9 years ago
r52226 / r52227
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

9 years ago
Created attachment 402468 [details]
Post-fix screenshot
(Reporter)

Comment 5

9 years ago
Verified FIXED.
Status: RESOLVED → VERIFIED

Updated

9 years ago
Whiteboard: aaq → aaq sumo_only
You need to log in before you can comment on or make changes to this bug.