Closed
Bug 911955
Opened 11 years ago
Closed 11 years ago
[WAP push] Wap push manager displays message with other content types more than just SI and SL.
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(blocking-b2g:koi+)
VERIFIED
FIXED
blocking-b2g | koi+ |
People
(Reporter: echu, Assigned: gsvelto)
Details
(Whiteboard: [FT:RIL])
Attachments
(4 files)
WAP push manager for user story bug 891248 still displays content type "text/vnd.wap.connectivity-xml" and "application/vnd.wap.connectivity-wbxml" message. It should only handle SI(text/vnd.wap.si)and SL(text/vnd.wap.sl) message. * Build Number Gaia: 9fb5802df60a9081846d704def01df814ed8fbd4 Gecko: http://hg.mozilla.org/mozilla-central/rev/b6c29e434519 BuildID 20130901040215 Version 26.0a1 * Reproduce Steps Send content type "text/vnd.wap.connectivity-xml" and "application/vnd.wap.connectivity-wbxml" messages to DUT. * Expected Result no wap push message notification shows. * Actual Result shown in wap push message notification. * Occurrence rate 100%
Updated•11 years ago
|
blocking-b2g: koi? → koi+
Updated•11 years ago
|
Assignee: nobody → gsvelto
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•11 years ago
|
||
Trivial patch that drops all messages whose content-type is not "text/vnd.wap.si" or "text/vnd.wap.sl".
Attachment #802398 -
Flags: review?(21)
Attachment #802398 -
Flags: review?(21) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Pointer to Github pull-request
Assignee | ||
Updated•11 years ago
|
Attachment #802853 -
Attachment description: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/12102 → [PULL REQUEST] Discard all non-SI/SL WAP Push messages
Assignee | ||
Comment 4•11 years ago
|
||
Merged to master: https://github.com/mozilla-b2g/gaia/commit/0f33ab65f6766818b296f3d272f1e430ae973d0d
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Verify on following buid Gaia: e58cfdac65fe644dc79fb3613fdaa9d9573537ac1378985810 Build: 20130912134644
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•