Commit Graph

174 Commits

Author SHA1 Message Date
Gregory Szorc
9b928cab7a Bug 1055102 - Properly handle Unicode in Bagheera payloads; r=bsmedberg
It was observed that FHR was sending invalid JSON payloads to the
server. Specifically, JSON payloads contained invalid Unicode strings.

Investigation revealed that the culprint was CommonUtils.convertString()
silently swallowing high bytes. When the Bagheera client went to gzip
the JSON payload, the input buffer into gzip was missing high bytes.

This patch changes the bagheera client to UTF-8 encode strings before
gzip, thus ensuring all data is preserved. A corresponding change was
also added to the mock bagheera server implementation.

Alternatively, we could have changed CommonUtils.convertString() to
be high byte aware. However, many consumers rely on this function.
This patch is written with the intent of being uplifted and the change
performed is targeted at the specific problem.

Tests for Unicode preserving behavior have been added to both the
generic Bagheera client and to FHR. The latter test is arguably
not necessary, but peace of mind is a good thing, especially with
FHR.

See also bug 915850.

--HG--
extra : rebase_source : 4efddea7767c2e5f8cf19df247c3aba07c40eec6
extra : amend_source : ae3b6d89efa54fc9ed1794404476622946ad4b22
2014-08-19 09:12:12 -07:00
Nathan Froyd
ae562efb3f Bug 1045617 followup - remove dummy mach command DONTBUILD; r=me 2014-08-06 12:28:37 -04:00
Nathan Froyd
072b3b906e Bug 1045617 - move services's run_server.py Makefile rules into mach; r=gps 2014-07-30 15:44:13 -04:00
Nathan Froyd
08483fe6ee Bug 1054476 - move INSTALL_TARGETS in services/ Makefiles to EXTRA_{PP_,}JS_MODULES; r=gps 2014-07-28 15:24:01 -04:00
Nathan Froyd
be58364116 Bug 1054476 - use PREF_JS_EXPORTS in services/ Makefiles; r=gps 2014-07-28 14:42:26 -04:00
Birunthan Mohanathas
e3009122fa Bug 1048989 - Fix 'useless expression' warnings with MERGED_COMPONENT. r=gps 2014-08-07 21:52:05 -07:00
Steve Workman
20fec6e951 Bug 982816 - Remove extra calls to nextTest in test_hawkrequest.js r=ckarlof 2014-07-08 11:44:03 -07:00
Gregory Szorc
54c4d3b9ae Bug 1033836 - Convert TESTING_JS_MODULES to moz.build; r=glandium
--HG--
extra : rebase_source : 1cd9e402c507260358af7a93c6fb2fc91c24994b
2014-07-02 16:43:41 -07:00
Gregory Szorc
cd3dbd7747 Bug 1033836 - Rename services-common to services/common; rs=rnewman
The upcoming build system patches don't support hypthens in path names.
Changing this for that reason is kind of silly, but it's the easiest
way. Besides, nothing else uses hyphens in directory names.

--HG--
extra : rebase_source : 42dda2b1f16a3c0bfe17397a70092362e400530f
2014-07-02 16:49:36 -07:00
Fernando Jiménez
7746ac6a1d Bug 1027595 - TypeError: deriveHawkCredentials is not a function after bug 1020859. r=jedp 2014-06-21 07:43:39 +01:00
Mark Banner
6eb2c9598f Bug 1020859 Part 1 - Make HawkClient return all the response details for a request, and make deriveHawkCredentials common code. r=jparsons 2014-06-18 10:42:15 +01:00
Mark Hammond
98316383b5 Bug 1024812 - unhandled exceptions in hawkclient no longer cause sync to hang. r=rnewman 2014-06-14 11:47:34 +10:00
Mark Hammond
3436d9c264 Bug 1015794 - use Object.create to create services timer objects. r=rnewman 2014-06-12 10:06:47 +10:00
Fernando Jiménez
f896544f4d Bug 988469 - MSISDN verification API for privileged apps. Part 2: Make FxA specific stuff generic. r=markh 2014-06-07 19:30:18 +02:00
Mark Hammond
e464cf75f7 Bug 1010531 - Prevent async callback from throwing when a callback is made with an error. r=rnewman 2014-05-28 10:00:00 +10:00
Mark Hammond
250befa6d2 Bug 1015794 - avoid mutating the prototype of a timer object to prevent warnings. r=rnewman 2014-05-28 09:54:52 +10:00
Mark Hammond
8213b883dc Bug 1006943 - non-sensitive FxA log info is included with sync logs. r=rnewman 2014-05-15 12:00:59 +10:00
Ehsan Akhgari
4c3acbbb4f Bug 1005710 - Avoid preprocessing the services JS exports files needlessly; r=mshal 2014-05-05 21:28:55 -04:00
Irving Reid
8797f73681 Bug 966674: #2 Move exceptionStr() and stackTrace() from utils.js to Log.jsm; r=gps,rnewman 2014-04-29 10:09:27 -04:00
Mark Hammond
bc93fc2b9d Bug 990834 (part 2) - Add support/tweak retry and backoff header support to hawk and tokenserverclient. r=rnewman 2014-04-10 12:08:19 +10:00
Marco Castelluccio
8a53b11c8e Bug 993173 - Use encoding option in readJSON. r=Yoric 2014-04-08 04:47:00 +02:00
Mark Hammond
3d5d088ab8 Bug 985902 - expose fxaccounts/MOZ_SERVICES_FXACCOUNTS to mozinfo and only run test_hawkclient when enabled. r=ted 2014-03-31 14:15:05 +11:00
Mark Hammond
40e12f4211 Bug 985212 - check if the username looks like an email address to determine what sync to use. r=rnewman 2014-03-24 11:44:43 +11:00
Phil Ringnalda
08f929b520 Back out b9597187a3d6 (bug 985212) for xpcshell bustage
CLOSED TREE
2014-03-22 21:30:46 -07:00
Mark Hammond
b52785af62 Bug 985212 - check if the username looks like an email address to determine what sync to use. r=rnewman 2014-03-23 14:20:38 +11:00
Phil Ringnalda
44d24e0d0d Back out 61454adff1d6 (bug 985212) for xpcshell bustage 2014-03-20 21:50:21 -07:00
Mark Hammond
0ebee1740a Bug 985212 - check if the username looks like an email address to determine what sync to use. r=rnewman 2014-03-21 14:39:35 +11:00
Richard Newman
b6c49a6b1c Bug 978876 - Part 0: add set utilities to CommonUtils. r=gps 2014-03-13 16:37:25 -07:00
Mark Hammond
dbd111e61c Bug 958447 - respect Retry-After header from token server. r=rnewman 2014-03-12 19:27:22 -07:00
Jed Parsons
639e1d6ba2 Bug 969892 - Fixes and improvements to FxAccounts test suite; r=ttaubert 2014-02-18 09:47:52 -08:00
Mark Hammond
bce6e55f5d Bug 977502 (part 2) - add tests with mock http responses simulating errors. r=ckarlof 2014-03-07 15:41:32 +11:00
Mark Hammond
56e312c5de Bug 976393 - ensure authentication errors log as much detail as possible. r=jedp 2014-03-03 09:55:30 +11:00
Mark Hammond
fb5728ab42 Bug 977931 - return an error object even when restResponse.headers is null. r=spenrose 2014-02-28 13:24:49 +11:00
Sam Penrose
28968a9820 Bug 945449 - FxAccountsClient should support auth server's backoff protocol. r=ferjm 2014-02-26 10:11:49 -08:00
Jed Parsons
d103f216bf Bug 974990 - hawk request to access lang prefs as infrequently as possible. r=rnewman
--HG--
rename : services/common/hawk.js => services/common/hawkclient.js
rename : services/common/tests/unit/test_hawk.js => services/common/tests/unit/test_hawkclient.js
2014-02-25 09:19:47 -08:00
Geoff Brown
9a962e2456 Bug 969624 - Skip test_storageservice_client.js on Android 2.3 emulator; r=dminor 2014-02-24 17:51:15 -07:00
Sam Penrose
a1fc765c95 Bug 955953 - FxAccountsClient should set Accept-Language header on all server calls. r=rnewman 2014-02-16 08:39:05 -08:00
Gregory Szorc
1290f96f1d Bug 972093 - Detect run_next_test() during add_task(); r=ted, rnewman
If an xpcshell test called run_next_test() from inside an add_task(),
bad things would happen. This patch detects that behavior and aborts the
test immediately with an actionable error message.

--HG--
extra : rebase_source : c5cf11070421b95a6fcf476d0a86289a3d0ba73a
extra : amend_source : cec6ad2a751bba01e7d4ecbff0735b39e3326efb
2014-02-12 16:47:29 -08:00
Jed Parsons
94f10be0d7 Bug 967372 - Fix intermittent test_hawk.js failures on Windows. r=ttaubert 2014-02-08 17:07:34 -08:00
Gregory Szorc
2075042f8b Bug 970336 - test_restrequest.js times out on Android 2.3 emulator; r=rnewman
--HG--
extra : rebase_source : 686beb0c5ccfab3ebde27f554a3fa60439df5e53
extra : amend_source : 787e95b977f4557073f8c30b9e4e97269e6d1572
2014-02-12 17:16:44 -08:00
Mark Hammond
49015f9bf8 Bug 965896 - allow a sync startOver to re-enable Firefox Accounts. r=rnewman 2014-02-12 17:25:41 +11:00
Jed Parsons
95f73416a1 Bug 943521 - Use onepw prototcol in fxa client. r=ckarlof 2014-02-04 22:14:30 -08:00
Ryan VanderMeulen
085446242d Backed out changeset 79b198be3e50 (bug 943521) for xpcshell failures. 2014-02-07 17:55:12 -05:00
Jed Parsons
1ab63753b4 Bug 943521 - Use onepw prototcol in fxa client. r=ckarlof 2014-02-04 22:14:30 -08:00
Peiyong Lin
6d3ae7dcf8 Bug 964735 - Convert libc into Library to make it link lazily. r=Yoric 2014-02-07 10:37:21 -05:00
Fernando Jiménez
d82eee2f25 Bug 967503 - Make HawkClient work in b2g. r=rnewman 2014-02-04 20:32:16 +01:00
Tim Taubert
8204a4c5b7 Bug 964922 - Enable FxA Sync on Nightly r=markh 2014-01-30 12:09:12 -08:00
Chris Karlof
71b8861505 Bug 966560 - fix bugs with time skew fix in bug 957863; r=jedp 2014-01-31 16:43:36 -08:00
Ryan VanderMeulen
a70bcc0e9f Backed out changeset cc1bac015920 (bug 966560) for xpcshell failures. 2014-01-31 21:00:47 -05:00
Chris Karlof
baef89768e Bug 966560: fix bugs with time skew fix in bug 957863, r=jedp 2014-01-31 16:43:36 -08:00