-
Notifications
You must be signed in to change notification settings - Fork 140
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
Fix Node 14 Compatibility #1361
Closed
JacksonWeber
wants to merge
133
commits into
microsoft:main
from
JacksonWeber:jacksonweber/make-node-14-compat
Closed
Fix Node 14 Compatibility #1361
JacksonWeber
wants to merge
133
commits into
microsoft:main
from
JacksonWeber:jacksonweber/make-node-14-compat
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add eslint validation * Fix * Update dependencies * Run only on node 14 * WIP
* Statsbeat do not count failed request when throttle * Fix responde code for throttling * Add new status code handling
* Bunyan error not sent correctly * package.-lock
* Shorten host property in network statsbeat * Addressing comments
* Validate compatibility with oher versions of node.js runtime * Test * Test * Update versions * Add node 16 in supported matrix
* Adding web snippet injection * Update test * Addressing comments * Updating tests * Addressing comments * Fixing test, updatign env variable to new one * web snippet auto injection * add test snippet npm package * remove extra input * updates * add tests * fix eslint * remove debug snippet * merge uodates Co-authored-by: Hector Hernandez Guzman <hectorh@microsoft.com> Co-authored-by: Hector Hernandez <39923391+hectorhdzg@users.noreply.github.com>
* add snippet injection readme * fix typo * add readme defaults
Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6. - [Release notes](https://github.com/substack/minimist/releases) - [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6) --- updated-dependencies: - dependency-name: minimist dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Hector Hernandez <39923391+hectorhdzg@users.noreply.github.com>
Bumps [ansi-regex](https://github.com/chalk/ansi-regex) from 3.0.0 to 3.0.1. - [Release notes](https://github.com/chalk/ansi-regex/releases) - [Commits](chalk/ansi-regex@v3.0.0...v3.0.1) --- updated-dependencies: - dependency-name: ansi-regex dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Hector Hernandez <39923391+hectorhdzg@users.noreply.github.com>
… dispose+setup after http.createServer (microsoft#947) * fix(AutoCollectHttpRequests): Http requests aren't collect if we called dispose and setup after http.createServer * refactor(AutoCollectHttpRequests): Add underscore on private function * fix(Tests-EndToEnd): Http requests handler isn't placed on stub createServer function * refactor(AutoCollectHttpRequests): Set private the static property
…icrosoft#946) * Ensure dependencies are not tracked if disabled after initialization * Addressing comments
* WIP * WIP * Adding tests * Package-lock.json * Addressing comments
* 2.3.2 release * Update
* Fix issue with Statsbeat binding on shutdown * Test
* 2.8.0 release * Update * Ensure live metrics client is available * Revert * Force update
…osoft#1219) * Check for defaultClient to be present to avoid undefined errors * Update
* 2.9.0 release * Update package-lock.json * Update package-lock.json
* Handle Linux environments when checking if the agent attached already. * Update package-lock.json
A minimal fix for trying to load the default configuration during runtime which only exists in the source code. That loading previously failed when the code is bundled. So now the default configuration is already loaded when bundling the code. Co-authored-by: Hector Hernandez <39923391+hectorhdzg@users.noreply.github.com>
* Add new features to statsbeat. * Clean up native metrics statsbeat logic and add tests. * Update NativePerformance.tests.ts * Update NativePerformance.tests.ts * Ensure that undefined client/statsbeat will not error out. * Update statsbeat value for native metrics. * Update JsonConfig tests as overloaded readFileSync method is not attaching spy correctly.
* Update Web Snippet enum. * Remove web snippet.
…1267) * Add process ID to created file name for logs caching. * Test * Update file cleanup to only clean files linked to the PID. * Update EndToEnd.tests.ts * Update error handling in the file system helper. * Update FileSystemHelper.ts * Address PR comments. * Remove PID. * Update EndToEnd.tests.ts
* Update statsbeat values to be consistent with the exporter. * Update statsbeat tests.
* Update how we set sdk version in the agent and update tests. * Update Context.tests.ts
* Update how manual attach is checked. * Update Default.ts
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Node.js 14 compatibility is impacted by the Azure SDK. Conditionally load azure SDK packages to avoid breaking for customers using versions of Node that aren't supported by the Azure SDK.