Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(release): 1.17.2 #535

Merged
merged 15 commits into from
Apr 9, 2024

Conversation

shubhsheth
Copy link
Contributor

See CHANGELOG.md


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

qhanam and others added 11 commits February 12, 2024 09:23
Bumps [ip](https://github.com/indutny/node-ip) from 1.1.5 to 1.1.9.
- [Commits](indutny/node-ip@v1.1.5...v1.1.9)

---
updated-dependencies:
- dependency-name: ip
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ervability#525)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.4 to 1.15.6.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.4...v1.15.6)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…-observability#528)

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) from 6.0.1 to 6.1.2.
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v6.1.2/CHANGELOG.md)
- [Commits](webpack/webpack-dev-middleware@v6.0.1...v6.1.2)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: retry with exponential backoff

* fix

* test
* feat: limit PutRumEvents retry to 5xx

* feat: add 429

* cleanup

* cleanup

* docs

* doc

* fix: fmt
…y#531)

Bumps [express](https://github.com/expressjs/express) from 4.18.1 to 4.19.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.18.1...4.19.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: disableOnFail

* doc: add config doc

* Revert "doc: add config doc"

This reverts commit ec1f74f.

* Revert "feat: disableOnFail"

This reverts commit 67ed7b3.

* feat: only disable RUM when dispatch fails with 403 or 404

* chore: add unit tests

* feat: add 401
* fix: Ignore URL construction error from invalid performance resource event

* fix: Throw error when URL construction fails for invalid performance resource event

* fix: Ignore error thrown from URL construction

* test: add unit test

* fix: record resources with invalid names

* fix: Update unit test for invalid url

* fix: Update hostname typo in isPutRumEventsCall tests

---------

Co-authored-by: Billy <billyzh@amazon.com>
Sync changes from main for 1.17.2 patch release
@williazz
Copy link
Contributor

williazz commented Apr 3, 2024

issue: this patch should only contain #532

shubhsheth and others added 4 commits April 4, 2024 10:57
Revert "Sync changes from main for 1.17.2 patch release"
* fix: Ignore URL construction error from invalid performance resource event

* fix: Throw error when URL construction fails for invalid performance resource event

* fix: Ignore error thrown from URL construction

* test: add unit test

* fix: record resources with invalid names

* fix: Update unit test for invalid url

* fix: Update hostname typo in isPutRumEventsCall tests

---------

Co-authored-by: Billy <billyzh@amazon.com>
@shubhsheth
Copy link
Contributor Author

Updated the PR to only include #532

@shubhsheth
Copy link
Contributor Author

Hi @williazz and @qhanam, can I get a review on this PR?

Copy link
Contributor

@williazz williazz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Shubh

@williazz williazz merged commit 5d05cae into aws-observability:release/1.17.x Apr 9, 2024
3 checks passed
@shubhsheth
Copy link
Contributor Author

Thanks @williazz! Do we need to do anything additional for the changes to create a new version for the npm package, or is that automatic?

@williazz
Copy link
Contributor

williazz commented Apr 9, 2024

No I'll run the release process

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants