Граф коммитов

246 Коммитов

Автор SHA1 Сообщение Дата
Matthew Podwysocki 769c1b126e [EngSys] Update name of @azure-tools/test-utils ***NO_CI*** 2024-04-25 13:14:33 -07:00
Chenyang Liu 37f863c2ef
[webpubsubclient] Add keepalive and quick ack (#29353)
### Packages impacted by this PR
@azure/web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR
This PR try to resolve 2 kinds of issues:
1. The original logic returns sequenceAck every second but if there're a
lot of receiving messages, the client may fail to ack in time (service
only buffer 1000 unacked messages).
2. In some cases, the ws may in the half close state (service has closed
the connection but client don't recognize it). The websocket library
doesn't proactively send ping, and results in some cases that the
connection may wait for TCP timeout which is too long.

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
1. We quick ack if we have more than 300 messages unacked
2. Add a timer to send something, it can trigger TCP to detect closure
quickly

### Are there test cases added in this PR? _(If not, why?)_
Yes

### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2024-04-24 09:42:43 +08:00
Jeremy Meng e9561c1a52 [EngSys] back to use `nyc` for code coverage
now that we removed `esm` dependency, `nyc` should be working again, and `nyc`
appears to have better results than `c8` does.

***NO_CI***
2024-04-19 12:19:32 -07:00
Maor Leger 224f7c1c14 [EngSys] Migrate to dev-tool extract-api ***NO_CI*** 2024-04-19 18:13:41 +00:00
Maor Leger 446e9d831b [EngSys] Remove ESM dependency ***NO_CI*** 2024-04-19 15:45:48 +00:00
Jeremy Meng ffe81c5bcf [EngSys] upgrade dev dependency `mkdirp` to `^3.0.1`
***NO_CI***
2024-04-15 11:27:58 -07:00
Jeremy Meng 398494acd8 [EngSys] upgrade dev dependency `typescript` to `~5.4.5`
The only package that still stays on ~5.3.3 is search-document (tracked by issue #29283)

***NO_CI***
2024-04-11 23:19:38 +00:00
Maor Leger d7173879bf [EngSys] C8 major version update ***NO_CI*** 2024-03-22 13:26:43 -05:00
Maor Leger e3aaa5abef
remove use-esm-workaround (#28826)
Builds off of Matt's work on moving to tsx in #28801 by removing the
`use-esm-workaround` flag from packages that needed it before we moved to tsx.

There's additional cleanup to be had, but I am trying not to cause a
build storm.

We are at a point where we can delete `esm` globally!

Contributes to #28617 which can be completed with a no-ci change to
remove `esm` globally

****NO_CI****
2024-03-12 12:32:59 -07:00
Scott Beddall (from Dev Box) ecdb45e61d replace all tests.yml usage w/ extends to archetype-sdk-tests.yml ***NO_CI*** 2024-03-11 14:13:06 -07:00
Matthew Podwysocki ca72dcf5c9 [EngSys] move to rimraf 5 ***NO_CI*** 2024-03-07 16:48:30 -08:00
Maor Leger 4415ff81a7
[EngSys] Bump puppeteer to 22.2.0 (#28639)
Looks like Google updated their URLs for chrome for testing downloads,
planning to retire the `hdgedl.me.gvt1.com...` URLs with
`storage.googleapis.com` URLs

It may have been deprecated for quite some time, but looks like as of
last night the old URLs have died, returning 500 instead.

As a result, puppeteer postinstall breaks the build

According to https://github.com/puppeteer/puppeteer/issues/11967
puppeteer 22.2.0 already uses the new URLs which
we can easily upgrade to.

This PR bumps the min-version to 22.2.0 across the board.
2024-02-22 18:49:19 +00:00
Jeremy Meng cd4d6b9f6f
[EngSys] switch to use `esm4mocha` loader by default for *.js tests (#28589)
- Switch to use esm4mocha loader by default
- Add `--use-esm-workaround=true` to packages that need more work
(mostly due to usage of `__dirname` that is no longer available in ESM)
- Change some usage of `__dirname` to `"."` when possible
- [esm4mocha] only transform modules whose format is changed
- [load-testing-rest] update tsconfig.json to be consistent with other
packages
2024-02-16 14:37:24 -08:00
Jeremy Meng 09784c3724 [EngSys] upgrade dev dependency `pretter` to `^3.2.5`
There's some changes in 3.2.5 which caused formatting changes for tsconfig.json
which caused our rush automation failed in check-format when moving to latest 3.2.5.

https://github.com/prettier/prettier/blob/main/CHANGELOG.md#use-json-parser-for-tsconfigjson-by-default-16012-by-sosukesuzuki

This change upgrades prettier to version ^3.2.5 and format files using "rush format".

***NO_CI***
2024-02-06 12:13:14 -08:00
Jeremy Meng 18d6d50c67
[EngSys] upgrade dev dependency `puppeteer` to `^22.0.0` (#28200)
by running `rush add -m --dev -p puppeteer@^22.0.0`

- adjust core-xml test due puppeteer switched from Chromium to Chrome
- ignore Accept-Language headers for browser playback tests

Resolves https://github.com/Azure/azure-sdk-for-js/issues/25790
2024-02-06 12:05:36 -08:00
Maor Leger c0d505f086 [EngSys] Update to @azure/identity 4.x
This change migrates every package to the latest major version of
identity in order to ensure we can catch any regressions earlier in the
development process.

I chose to upgrade the samples' package.json as well - but happy to
revert that if there are objections.

***NO_CI***
2024-02-02 19:01:35 +00:00
Jeremy Meng fb74e0b24b [EngSys] ensure dev dependency `@types/mocha` when depending on `mocha`
Our tests use the type `Mocha.Context` when they need to pass the test context
to test recorder. However, without a dev dependency on `@types/mocha`
compilation may fail when recorder constructor is changed from taking a
`Mocha.Test` to taking an interface that is compatible.

This change ensures the dev depedency on `@types/mocha` too if there's a dev
dependency on `mocha`.

***NO_CI***
2024-02-02 18:41:28 +00:00
Jeremy Meng ee5c29d72d ***NO_CI***
bump to ~5.3.3
2024-01-26 11:25:48 -08:00
Matthew Podwysocki f70fa9821c [EngSys] Update TypeScript to 5.3 2024-01-26 11:25:47 -08:00
Jeremy Meng 0e1c8881f0 [EngSys] upgrade dev dependency `prettier` version to ^3.2.4
***NO_CI***

Latest versions of prettier now correctly formats `tsconfig.json`, which results
in differences with previous version of 3.1.1 in pnpm-lock.yaml. This is failing
the automated rush update pipeline because `check-format` fails with version
3.2.4.

This includes changes of
- bumping prettier version to ^3.2.4
- running `rush format`
2024-01-18 19:05:41 +00:00
Jeremy Meng 12b4dcc257 [EngSys] upgrade to prettier v3
***NO_CI***

- remove dev dependency `prettier` from non-tool packages when possible and
  update them to run vendored prettier for `check-format` and `format` scripts

- upgrade rest of packages to prettier v3

- run `rush format` for all rush packages

This is a follow-up to https://github.com/Azure/azure-sdk-for-js/pull/28127 to
move the rest of repo to prettier v3.
2024-01-10 18:45:01 -05:00
Azure SDK Bot 24f754174e
Post release automated changes for web-pubsub releases (#28131)
Post release automated changes for azure-web-pubsub-client
2024-01-03 13:41:36 -06:00
Jeremy Meng f6e6736d3a [EngSys] remove dev dependency `mocha-junit-reporter`
***NO_CI***

We have replaced it with Mocha builtin XUnit reporter in PR https://github.com/Azure/azure-sdk-for-js/pull/27992
2023-12-06 22:28:15 +00:00
Jeremy Meng 02491d75c2
[EngSys] upgrade dev dependency `rollup` version to `^4.0.0` (#27997)
### Issues associated with this PR
#27450
2023-12-06 17:43:06 +00:00
Chenyang Liu 9a0d1f7862
Update and prepare GA for @azure/web-pubsub-client (#27694)
### Packages impacted by this PR
@azure/web-pubsub-client

### Describe the problem that is addressed by this PR
Update according to GA API review:
https://apiview.dev/Assemblies/Review/492267bc4ab44b19a558635733047147

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-11-29 21:49:52 -05:00
Jeremy Meng facc44641b
[EngSys] upgrade dev dependency @types/sinon to ^17.0.0 (#27836)
and sinon too to ^17.0.0 for packages that missed last upgrade.
2023-11-17 14:54:01 -08:00
Azure SDK Bot ad08930eaf
Post release automated changes for web-pubsub releases (#27616)
Post release automated changes for azure-web-pubsub-client-protobuf
2023-11-09 13:17:30 -08:00
Jeremy Meng c6dbe7ef23
[EngSys] upgrade dep dependency `sinon` version to `^17.0.0` (#27601) 2023-11-02 17:14:17 +00:00
Jeremy Meng 09111131ef [EngSys] Increment minimum supported node version to v18
***NO_CI***

- React to @types/node changes
- Fix eslint-plugin tests
2023-10-30 21:01:37 +00:00
Jeremy Meng 15d1080408
[Doc] fix broken links to Node.js releases (#27537)
A couple of README files have different link than most packages' in this
repo, likely due to in-flight PRs when we update the link to point to
github one. These are now broken. This PR replaces them with the one
used in other packages.
2023-10-25 16:28:59 +00:00
kazrael2119 5dc36a720b
[mgmt] webpubsub release (#27366)
https://github.com/Azure/sdk-release-request/issues/4618
2023-10-25 16:50:25 +08:00
Jeremy Meng 4371e976b1 [engsys] replace dev dependency `nyc` with `c8`
***NO_CI***

The combination of `nyc` + `esm` is broken in latest versions of
NodeJS (https://github.com/istanbuljs/nyc/issues/1530#issuecomment-1773403365).
Since `esm` package is no longer actively maintained and its repo is archived,
it's less likely that the issue will be fixed soon.

This change switches to use another code coverage tool `c8` which is not
affected. `c8` respects `.nycrc` config files so those are not renamed.
2023-10-23 17:08:26 -04:00
Jeremy Meng 6d7f6359cb [engsys] update dev dependency `typescript` version to `~5.2.0`
***NO_CI***

- Upgrade typedoc too as older version only supports TypeScript 5.0
2023-10-23 14:32:05 -04:00
Chenyang Liu e2d0a5e784
Prepare for beta.2 release (#27379)
### Packages impacted by this PR
@azure/web-pubsub-client-protobuf

### Describe the problem that is addressed by this PR
Update Changelog for preparing beta.2

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-10-18 15:26:36 -07:00
Deyaaeldeen Almahallawi 72d460d5ba Use dev-tool bundle in arm libraries ***NO_CI*** 2023-10-16 21:03:26 +00:00
Deyaaeldeen Almahallawi 2a896139d4
Upgrade to rollup v3 (#27377)
### Packages impacted by this PR

[@azure/web-pubsub](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3154068&view=results)

[@azure/ai-form-recognizer](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3154070&view=results)

[@azure/communication-identity](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3154080&view=results)

[@azure/core-amqp](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3154074&view=results)
api management libs

### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/pull/27285

### Describe the problem that is addressed by this PR
Upgrading rollup to v3

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
N/A

### Are there test cases added in this PR? _(If not, why?)_
N/A

### Provide a list of related PRs _(if any)_
N/A

### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-10-11 13:17:02 -07:00
Chenyang Liu f10a423a1d
[web-pubsub-client] ParseMessages can return array of messages (#27345)
### Packages impacted by this PR
@azure/web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR
Change the interface of `parseMessages` and allow protocol implementor
to return a list of messages. It's designed for further possible new
protocols that contains multiple messages in one websocket frame.

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_
yes

### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-10-10 15:34:46 -07:00
Jeremy Meng c65d843a30 Increment minimum supported node version to 16
***NO_CI***

- update engine.node to >=16.0.0 in package.json
- update @types/node version to ^16.0.0
- update dev-tool sample's MIN_SUPPORTED_NODE_VERSION to 16
- update eslint-plugin's rules and tests related to engine.node version
- remove TextEncoder and TextDecoder stubs as they are now on global object
- fix tests compiler error due to better typings in v16
- update some README files to not reference version like 14.x.x
2023-10-09 21:09:54 +00:00
Deyaaeldeen Almahallawi c86e72cb4c
[dev-tool] Upgrade to rollup v3 (#27285)
### Packages impacted by this PR
All libraries

### Issues associated with this PR
Fixes https://github.com/Azure/azure-sdk-for-js/issues/26665

### Describe the problem that is addressed by this PR
We need to upgrade Rollup to v3 so our cjs bundles could import ES
modules among other goodness.

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
See the linked issue for discussion of the upgrade. Credit to @xirzec
for writing up the sourcemaps plugin in-house replacement!

### Are there test cases added in this PR? _(If not, why?)_
N/A

### Provide a list of related PRs _(if any)_
[N/A](https://github.com/Azure/azure-sdk-for-js/pull/27258)

### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-09-29 18:28:58 +00:00
kazrael2119 ab391b646b
[mgmt] update quick-start link in changelog (#27260)
https://github.com/Azure/azure-sdk-for-js/pull/27223#discussion_r1338094846
2023-09-27 17:51:50 +08:00
Chenyang Liu 8d8ded0775
[web-pubsub-client] React native support (#27241)
### Packages impacted by this PR
@azure/web-pubsub-client

### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/26400

### Describe the problem that is addressed by this PR
React native use some different packages from node, I added some package
map to resolve it.

### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-09-26 13:33:35 -07:00
Jeremy Meng b2dfb59d59 [test] upgrade dev dependency `mocha` to `^10.0.0`
- along with it, `@types/mocha` version to `^10.0.0`
- add `esm` dev dependency as they are used, but not explicitly list
- use dev-tool run command for test scripts as much as possible
- fix test issue caused by Mocha behavior change around test name
- move `import "chai/register-should"` to mocha -r command line option for core
  http tests

***NO_CI***
2023-09-26 18:54:14 +00:00
Chenyang Liu 7fbc63087c
Fix the bug that commonjs doesn't work for @azure/web-pubsub-client-protobuf (#27209)
### Packages impacted by this PR
@azure/web-pubsub-client-protobuf

### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/25703

### Describe the problem that is addressed by this PR
As protobufjs is purely commonjs type but rollup bundle didn't add
commonjs support. So only file in `dist-esm/` works but in `dist`
doesn't.

### Are there test cases added in this PR? _(If not, why?)_
Test already added `cjs()` plugin in bundle, so it works.
2023-09-26 09:12:57 -07:00
Deyaaeldeen Almahallawi e95422bbd0 Fix up Azure dependencies ***NO_CI*** 2023-09-22 18:38:37 +00:00
Jeremy Meng ddc5a19e0c [test][mgmt] add `testPollingOptions`/`updateIntervalInMs` of 0
***NO_CI***

so that LRO operations in playback mode don't wait.
2023-09-22 18:26:46 +00:00
Jeremy Meng ddb0950e0d
[dev-tool] add a `--no-test-proxy` option to run test commands (#26991)
It is desirable to centralize our NPM test scripts so that we can change
common options easily. Currently test-proxy is always launched, which
isn't useful for packages (e.g., core) that don't have recorded tests.

This PR adds an opt-in option to not launch test-proxy when running
tests.


### Packages impacted by this PR
`@azure/dev-tool`
2023-09-01 11:17:45 -07:00
Jeremy Meng 56d2330abe [EngSys] add `ts-node` dev dependencies to rush projects
***NO_CI***

Most of our packages, if not all, have dev dependency on `ts-node` either
directly (`mocha --require ts-node/register`) or indirectly via dev-tool (`run
test:node-ts-input`). Currently tests are running fine because mocha is able to
resolve ts-node currently. It may fail in other cases though (e.g., after
migrating a package to ESM).

This ensures `ts-node@^10.0.0` and `types/node` are included for our rush
packages.
2023-08-23 14:36:24 -07:00
Azure SDK Bot 25e947c6b4
Post release automated changes for web-pubsub releases (#26631)
Post release automated changes for azure-arm-webpubsub
2023-07-31 12:01:15 -07:00
Liangying.Wei f8ef82e113
Update README.md (#26667)
Fixes https://github.com/Azure/azure-sdk-for-js/issues/26656


### Packages impacted by this PR
@azure/web-pubsub-express

### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/26656
2023-07-31 10:55:08 -07:00
Jeremy Meng 0d9f89ac90
[engsys] upgrade dev dependency `sinon` and `@types/sinon` (#26622)
- update `sinon` version to `^15.0.0`. The breaking changes does not
affect our usages.
- upgrade `@types/sinon` to `^10.0.0`. This reveals test issue where
mocked tokens are not compatible with `AccessToken`. This PR fixes the
tests too.
2023-07-27 09:54:37 -07:00
kazrael2119 8e999fb5e0
webpubsub release (#26450)
https://github.com/Azure/sdk-release-request/issues/4322
Recording:
https://github.com/Azure/azure-sdk-assets/tree/js/web-pubsub/arm-webpubsub_0dce2a64a8
2023-07-25 15:51:21 +08:00
Azure SDK Bot b79f40e5e0
Post release automated changes for web-pubsub releases (#26378)
Post release automated changes for azure-web-pubsub-express
2023-07-11 12:30:24 -07:00
Siyuan Xing 74b6caaf6a
[WebPubSub] Adding new features for 20230701 (#26323) 2023-07-11 11:45:39 -07:00
Liangying.Wei 8cc0cfb7cf
Remove the abuse protection check since the service always validates … (#26333)
…the request


### Packages impacted by this PR
@azure/web-pubsub-express
2023-06-29 09:40:07 -07:00
kazrael2119 63bfb1cd0c
migrate sdk recording files 3 (#25679)
https://github.com/Azure/azure-sdk-for-js/issues/25410
2023-04-28 16:49:05 +08:00
Harsha Nalluru 7d551f5939
[Web Pubsub] Migrate recordings to the assets repo (#25607)
Onboarding to the asset sync workflow, to push the recordings to the
assets repo.
Following the guide
[ASSET_SYNC_MIGRATION.md](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/test-utils/recorder/ASSET_SYNC_MIGRATION.md)

Please understand the new
[#workflow-with-asset-sync-enabled](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/test-utils/recorder/ASSET_SYNC_MIGRATION.md#workflow-with-asset-sync-enabled)
to maintain your recordings from now on.
2023-04-21 14:27:25 -07:00
Jeremy Meng d3a06a7349 [EngSys] upgrade dev dependency `typescript` to `~5.0.0`
***NO_CI***

- update versions in package.json
- change a couple ^ version to ~ version because typescript doesn't follow semver
2023-04-18 23:04:26 +00:00
Azure SDK Bot 151cba2c29
Post release automated changes for web-pubsub releases (#25486)
Post release automated changes for azure-web-pubsub-express
2023-04-11 07:01:54 +00:00
Azure SDK Bot 6b8effe55a
Post release automated changes for web-pubsub releases (#25507)
Post release automated changes for azure-web-pubsub-client
2023-04-11 07:01:22 +00:00
Chenyang Liu a9aeb7a350
[WebPubSubClient] Prepare release for beta.3 (#25503)
### Packages impacted by this PR
@azure/web-pubsub-client

### Issues associated with this PR
#24901

### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-04-10 04:49:11 +00:00
Liangying.Wei ab214aa047
Fix the query property inconsistency with the protocol (#25434)
### Packages impacted by this PR
@azure/web-pubsub-express

### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/24515

### Describe the problem that is addressed by this PR
The Azure Web PubSub service defines it's protocol sending CloudEvents out as `query`, however in TS we defined it as `queries` in the interface. To resolve the inconsistency between them, add a `query` property to the interface but mark it as obsolete and mean why set the value of `queries` as the value of `query` to make the property work.

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-04-06 02:32:41 +00:00
Azure SDK Bot 99a90e4211
Post release automated changes for web-pubsub releases (#25387)
Post release automated changes for azure-web-pubsub-client-protobuf
2023-03-29 08:13:51 +00:00
Chenyang Liu c0c05d9a43
Prepare for beta.1 release (#25376)
### Packages impacted by this PR
@azure/web-pubsub-client-protobuf

### Issues associated with this PR


### Describe the problem that is addressed by this PR
Prepare for beta release

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-03-28 01:23:01 +00:00
Jeremy Meng 4e96139d92
[engsys] upgrade protobufjs dependencies (#25366)
- `protobufjs` to v7.2.2
- `protobufjs-cli` to v1.1.1

This is to address a security warning about indirect dependency of
`taffydb` (https://nvd.nist.gov/vuln/detail/CVE-2019-10790)
2023-03-27 13:56:39 -07:00
Chenyang Liu d67a1952c7
[web-pubsub-client] Fix isDuplicated always returned as true (#25239)
### Packages impacted by this PR
@azure/web-pubsub-client

### Issues associated with this PR
#24901
https://github.com/Azure/azure-sdk-for-js/pull/24902 try to fix this issue but blocked by format. 

### Describe the problem that is addressed by this PR
`IsDuplicated` always return true in WebPubSubResult.

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-03-27 05:16:50 +00:00
Azure SDK Bot e2540e430f
Post release automated changes for web-pubsub releases (#25301)
Post release automated changes for azure-arm-webpubsub
2023-03-20 20:01:45 +00:00
kazrael2119 e0deb7ef60
webpubsub release (#25178)
https://github.com/Azure/sdk-release-request/issues/3901
2023-03-15 16:05:23 +08:00
Chenyang Liu f32e42d43f
[web-pubsub-client-protobuf] Add more tests (#25183)
### Packages impacted by this PR
eb-pubsub-client-protobuf

### Issues associated with this PR


### Describe the problem that is addressed by this PR
Add more tests

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-03-14 14:30:22 +00:00
Chenyang Liu 4538a2818d
[webpubsub] Init pr for client protobuf protocol (#24879)
### Packages impacted by this PR
@azure/web-pubsub-client-protobuf

This is the init pr for @azure/web-pubsub-client-protobuf. Please review the package name, dependencies.

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-03-09 05:54:09 +00:00
Jeremy Meng ae26cc9d92 [engsys] remove karma ie and edge launcher
as we no longer need to test them.

***NO_CI***
2023-03-06 17:08:15 -05:00
Jeff Fisher b1509d6756 Update test-recorder to drop support for `@Azure/core-http` and
bump the major version accordingly.

***NO_CI***
2023-03-02 15:21:26 -08:00
Jeremy Meng 42d359f404 [engsys] upgrade @rollup plugins dev dependencies
***NO_CI***

- `@rollup/plugin-commonjs` to `^24.0.0`
- `@rollup/plugin-json` to `^6.0.0`
- `@rollup/plugin-multi-entry` to `^6.0.0`
- `rollup-plugin-polyfill-node` to `^0.12.0`
- `@rollup/plugin-inject` to `^5.0.0`
- `@rollup/plugin-replace` to `^5.0.0`

Fix notification-hubs rollup test config: now sourcemaps plugin is complaining
about .ts file. Move typescript plugin before it.

Fix service-bus to include needed rollup dependencies. It was lucky to be able
to re-use on other packages to install them but is failing in some builds.
2023-02-10 15:12:48 -08:00
Jeremy Meng ab536fc39a [engsys] make typescript version consistent to ~4.8.0
***NO_CI***

This is the result of

- `rush add --dev -m -p typescript@~4.8.0`

- and update versions in non-rush projects
2023-02-10 11:41:53 -08:00
Kevin Guo 8023ee9e7c
Organized and edited the README of Web PubSub client library (#24743)
Organized and edited README so that it's easier to get started with this client library and easier to find further reference docs.

### Packages impacted by this PR


### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-02-09 08:54:22 +00:00
Azure SDK Bot 5f728af73f
Post release automated changes for web-pubsub releases (#24737)
Post release automated changes for azure-web-pubsub-client
2023-02-06 19:38:19 +00:00
Chenyang Liu 0f001f7b3c
[web-pubsub-client] Prepare release for Beta.2 (#24647)
### Packages impacted by this PR


### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-02-06 03:18:41 +00:00
Jeremy Meng 61e10c73c9
[engsys][loadtesting,web-pubsub] upgrade dev dependency api-extractor to `^7.31.1` (#24619)
- unpin version in `load-testing-rest`. We should be using the latest like all
other packages.
2023-02-02 11:42:10 -08:00
Jeremy Meng 0664dab07a
[engsys] upgrade dev dependency `@types/node` to `^14.0.0` (#24653)
for packages that missed the last version bump train.
2023-02-01 12:40:10 -08:00
Chenyang Liu 719827bf54
[web-pubsub-client] Remove signalr term (#24626)
### Packages impacted by this PR
wpb-pubsub-client

### Issues associated with this PR
https://github.com/Azure/azure-webpubsub/issues/477

### Describe the problem that is addressed by this PR
Remove signalr term

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-02-01 08:08:58 +00:00
Chenyang Liu 1571b871ae
[web-pubsub-client] Fix naming restoreGroup and add tests (#24593)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
`autoRejoinGroup` options didn't take effect and some naming are `autoRestoreGroup` which doesn't keep consistent.

### Are there test cases added in this PR? _(If not, why?)_
yes

### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-01-30 02:32:03 +00:00
Jeremy Meng bbb29823c9 [engsys] upgrade dev dependency `dotenv` version to `^16.0.0`
***NO_CI***
2023-01-26 15:59:56 -08:00
Azure SDK Bot e77e4f0815
Post release automated changes for web-pubsub releases (#24471)
Post release automated changes for azure-web-pubsub-client
2023-01-13 21:29:36 +00:00
Chenyang Liu 29a670e44a
[web-pubsub-client] Prepare for beta release (#24452)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR
Prepare for beta release

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-01-12 09:29:55 +00:00
Chenyang Liu f2fa490692
[web-pubsub-client] Make ackable operation cancellable (#24363)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR
Make ackable operation cancellable

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_
Yes

### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2023-01-05 02:52:25 +00:00
Minh-Anh Phan e199238582
[web-pubsub-client] delete dependency on test recorder (#24316) 2023-01-04 15:22:23 -08:00
Azure SDK Bot 68ca43921e
Post release automated changes for web-pubsub releases (#24358)
Post release automated changes for azure-web-pubsub
2023-01-04 18:00:43 +00:00
Deyaaeldeen Almahallawi 2cc6829e3c
[Web PubSub] Patch release (#24354) 2023-01-03 16:31:44 -05:00
Deyaaeldeen Almahallawi a43aaf3fdf
Upgrade jsonwebtoken to address security vulnerability (#24340)
### Packages impacted by this PR
@azure/identity, @azure/web-pubsub, @azure/web-pubsub-express, and @azure/web-pubsub-client

Live tests runs:
- [Identity](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2081318&view=results)
- [webpubsub](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2081328&view=results)

### Issues associated with this PR
Fixes https://github.com/Azure/azure-sdk-for-js/issues/24337
Fixes https://github.com/Azure/azure-sdk-for-js/issues/24336
Fixes https://github.com/Azure/azure-sdk-for-js/issues/24334

### Describe the problem that is addressed by this PR
There is a security vulnerability in v8 of the jsonwebtoken library. However, I can't shake the vulnerable version off our lock file completely because @azure/msal-node depends on it and they didn't make a release with the upgraded dependency yet, [see my comment on their PR](https://github.com/AzureAD/microsoft-authentication-library-for-js/pull/5503/files#r1058120234).

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
N/A

### Are there test cases added in this PR? _(If not, why?)_
N/A

### Provide a list of related PRs _(if any)_
N/A

### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [x] Added a changelog (if necessary)
2023-01-03 12:40:19 -05:00
Chenyang Liu cdd9c529e0
[web-pubsub-client] Update readme (#24291)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR
Update Readme

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2022-12-21 07:53:13 +00:00
Chenyang Liu dc0dfb0122
[web-pubsub-client] Add more tests (#24259)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_
Yes. Add more tests.

### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2022-12-19 09:32:32 +00:00
Guillaume FORTAINE cbfc919f8e
[web-pubsub-client] Fix some bugs (#24202)
- fix(web-pubsub-client): fix browser export for ws
- fix(web-pubsub-client): various fixes in samples

<!-- Give your Pull Request a brief description, and answer the following questions. This helps us find the best reviewers for the PR. -->

### Packages impacted by this PR


### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR: _(Applicable only to SDK release request PRs)_


### Checklists
- [ ] Added impacted package name to the issue description.
- [ ] Does this PR need any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here.)_
- [ ] Added a changelog (if necessary).
2022-12-15 01:50:23 +00:00
Jeff Fisher 57ead9a6c4
[core/many] Unpin dependencies (#24206) 2022-12-13 21:56:09 +00:00
Jeremy Meng 7115313003
[engsys] update `puppeteer` version in `web-pubsub-client` (#24216)
to `^19.2.2` which is consistent with the rest of rush projects
2022-12-13 12:08:02 -08:00
Chenyang Liu c84784c1cd
[web-pubsub-client] Add a browser sample and fix some bugs (#24185)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2022-12-09 10:40:25 +00:00
Chenyang Liu 9be5107c6e
[WebPubSub] Initial PR for web-pubsub-client (#24029)
### Packages impacted by this PR
web-pubsub-client

### Issues associated with this PR


### Describe the problem that is addressed by this PR
This is the init PR for new package web-pubsub-client.

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_
Yes

### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [x] Added a changelog (if necessary)
2022-12-08 03:21:14 +00:00
Jeremy Meng 63cf6bead5
[engsys] upgrade dev dependency `puppeteer` to `^19.2.2` (#24152)
* upgrade dev dependency `puppeteer` to `^19.2.2`

* update pnpm-lock.yaml

* Fix failing tests

by changing them to verify `error.name`.  The error message from parsing an
incomplete JSON string has been improved in recent versions of chromium.
Unfortunately it doesn't match NodeJS version any more.
2022-12-07 13:15:35 -08:00
Azure SDK Bot 5f459e8504
Post release automated changes for web-pubsub releases (#23948)
Post release automated changes for azure-arm-webpubsub
2022-11-24 04:47:02 +00:00
Azure SDK Bot 7626f3e5da
Post release automated changes for web-pubsub releases (#23965)
Post release automated changes for azure-web-pubsub
2022-11-24 04:46:03 +00:00
Liangying.Wei e19274197f
[WebPubSub]Adding new features (#23635)
### Packages impacted by this PR
@azure/web-pubsub

### Issues associated with this PR
N/A

### Describe the problem that is addressed by this P
Support new swagger
1. Support sending with "filter" syntax
2. Support generating access url with initial groups
3. Add new API "removeConnectionFromAllGroups" 

### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [ ] Added a changelog (if necessary)
2022-11-18 02:57:21 +00:00
kazrael2119 d7061ba7a5
webpubsub release (#23751)
* webpubsub release

* update files

* update files
2022-11-08 16:00:03 +08:00