fix(common): remove extra & in http params (#34896)

Previous to this commit, HTTP params like `{ a: '1', b: [], c: '3' }` resulted in a request like `a=1&&c=3` (note the double &&).

The ideal fix would probably be to stringify these params to `a=1&b=&c=3` like we do for empty string values. But that might be breaking as some APIs may rely on the absence of the parameter.

This fixes the issue in a compatible way by just removing the extra and unnecessary `&`, resulting in `a=1&c=3`.

PR Close #34896

packages/common/http/src/params.ts

@@ -224,9 +224,15 @@ export class HttpParams {
     return this.keys()
         .map(key => {
           const eKey = this.encoder.encodeKey(key);
+          // `a: ['1']` produces `'a=1'`
+          // `b: []` produces `''`
+          // `c: ['1', '2']` produces `'c=1&c=2'`
           return this.map !.get(key) !.map(value => eKey + '=' + this.encoder.encodeValue(value))
               .join('&');
         })
+        // filter out empty values because `b: []` produces `''`
+        // which results in `a=1&&c=1&c=2` instead of `a=1&c=1&c=2` if we don't
+        .filter(param => param !== '')
         .join('&');
   }