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

New Relic iOS agent: Staging 7.5.3 #331

Merged
merged 28 commits into from
Dec 3, 2024
Merged

New Relic iOS agent: Staging 7.5.3 #331

merged 28 commits into from
Dec 3, 2024

Conversation

cdillard-NewRelic
Copy link
Member

No description provided.

cdillard-NewRelic and others added 28 commits August 22, 2024 08:22
* [7.4.7-rc.589] Staging Release

* do cocoapods, then swift pm

* push SPM change for 7.4.7

* Branching from the main for beta tests (#133)

* Branching from the main for beta tests

* This should fix the tvOS beta not being found

* Added tvOS app tests

* Added a step to the app unit tests to add a secret api key

* Fixed some typos

* [7.4.7] Production Release (#159)

* [create-pull-request] automated change

* Fix auto-prod PR to include Cocoapod spec update

---------

Co-authored-by: cdillard-NewRelic <[email protected]>
Co-authored-by: Chris Dillard <[email protected]>

* push log reporting back to working state

* Add entity.guid back into NRLogger payload.

* [7.4.8-rc.710] Staging Release

* Include Prod Podspec Job by default in productionDeploy

* Verbose logging to Loggin endpoint

* log sessionId on logs endpoint success

* ws fix

* push WIP for /connect config parse

* Code dynamic enable/disable of LogReporting based on connect response

* Remote logging config

* [create-pull-request] automated change

* Initial pass at handling 409 to reconnect and fetch new config

* Send Support Metric kNRMAConfigurationUpdated when Logging Config is updated

* Impl setLogEntityGuid

* feat: Remote Logging with manual set entityGuid and ingestKey feature complete.

* NR-220019: App Token auth

* NR-220019: Adds GHA for Bespoke Branch Based Build (#198)

* Add 100 logs test, fix levelToString for "level" attribute on log messages.

* NR-228761: Implement uploadQueue for log data, NRLoggerTests

* Logs fix

* Use correct endpoints for logging

* [7.4.9-rc.824] Staging Release

* Fixing a merge mistake

* [7.4.9-rc.827] Staging Release

* [create-pull-request] automated change

* NR-223195: Support metric for success and failure remote logging

* NR-227297: logErrorObject impl, NR-227300: logAttributes impl

* [7.4.10-rc.860] Staging Release

* [7.4.10-rc.863] Staging Release

* NR-251740 - DT headers to dataTaskWithRequest call

When adding support for the New Event System, headers were improperly added just to the mutable request, which first was not even used to create the NSURLSessionTask. The headers were added to the mutableRequest after the task was created, meaning that it was not part of the task.

* [7.4.10-rc.867] Staging Release

* [create-pull-request] automated change

* Log attributes

* [7.4.11-rc.909] Staging Release

* bump runSymbolTool v to 7.4.11

* [7.4.11-rc.917] Staging Release

* [create-pull-request] automated change (#235)

Co-authored-by: cdillard-NewRelic <[email protected]>

* Staging 7.4.12 (#252)

* NR-256157: Remove Sample flag from traceState header for dist tracing (#229)

* [iOS] NR-266202: Adjust CryptoKit sha1 -> sha1256 (#240)

* [iOS] NR-266202: Adjust CryptoKit sha1 -> sha1256

* NR-266202: update xample

* NR-261761: NSSecureCoding (#249)

* 7.4.12 version bump, Dsym-upload-Tools: A  couple missed things for dsym-upload-tools, targeting 7.4.12 for fixing this (#251)

* dsym-upload-tools-update-agent-v

* Bump iOS agent project o 7.4.12, Bump dsym-upload-tools to 7.4.12

* fix to 7.4.12

* [7.4.12-rc.971] Staging Release

* [create-pull-request] automated change

* [7.5.0-rc.1036] Staging Release

* [create-pull-request] automated change

* Use macos-13 and xcode 14.3 for now w prod

* rem codecov in prod for now

* [create-pull-request] automated change

* Bump main branch swift package tools to 5.9 (#294)

* bump v to 7.5.2

---------

Co-authored-by: cdillard-NewRelic <[email protected]>
Co-authored-by: mbruin-NR <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Mike Bruin <[email protected]>
Co-authored-by: mbruin-NR <[email protected]>
Co-authored-by: smalsam-newr <[email protected]>
Co-authored-by: smalsam-newr <[email protected]>
Co-authored-by: Steve Malsam <[email protected]>
* NR-305349 - NRMAAnalytics lazy initialization

Check if NRMAAnalytics is already initialized before blindly re-initializing it again.

* Start a new session after background

When coming out of the background, we want to be able to start a new session, with a new session start time. This enables that.
* NR-285004 adding a readme for the example app

* Added a link to the example app readme in the real readme

* Update README.md
… app hang, this change causes harvestNow to work the same way as general app harvest. (#290)
* NR-307059 trying to make tests less flakey

* Added comments for the new public NRLogger.h function

* Returning an error from the get data function

* Logging the error if there is one
* NR-313973: Add Instrumentation Insights to /log

* NR-313973: fix

* NR-313973: adjust NRLogerTests
Bumped agent version to 7.5.3
…AttributeNames (#318)

* NR-317126 Added the prefix list to use in the new event system attribute validator

* Fixed or added test for prefixes
…nce Criteria: (#323)

* Pass nativePlatform as instrumentation insights name, collector.name is hybrid paltform.

* Swap collector.name and instrumention.name according to new Acceptance Criteria.
…og level. (#325)

* NRLogger: fix the log:inFile methods to check for remote log level. Fix up NRLoggerTests

* Remove unused form of log:inFile
* NR-318119 adding the ability to collect the logs sent to stdout and stderr

* NR-318119 added a way to extract type of OSLog and fixed timestamp format

* Removed unneeded test

* NR-318119 Renamed the flag to include stderr and changed the log divider to \n\n

* Trying to get more accurate timestamps when one's not there

* Changed the log collector to use a pipe and added a unit test

* Restoring the logging sooner in the test to hopefully see failed test logs

* Trying to fix a test that only fails on github

* trying to fix test run on github

* It looks like github unit tests don't have oslog debug enabled

* Looks like this test doesn't work on github for watchOS, will need to look further into it

* Unit test changes

* Restoring the first delay in the test

* Trying to make the test less flakey

* Using sleep instead of the wait

* Adjusting the sleep value

* We need default priority or the collected logs don't maintain perfect accuracy

* NRLogger: Fix to honor remote log level and local log level correctly.

* Revert "NRLogger: Fix to honor remote log level and local log level correctly."

This reverts commit 28790a5.

* Changing the name and default status of the feature flag, only redirect if remote logging is enabled

* Forgot to change flag in unit tests

* Added a stress test WIP, created the pipes one at a time so we can close them if needed

* Removed the feature flag and only capture logs when the debugger isn't attached

* Updating the Xcode version used in main.yml

* trying to update to macos-14 in main.yml

* trying xcode 15.4

* Changed the requiredAttributes to work with the auto-collected logs better

* Added back the feature flag after discussion

* Update Agent/Utilities/NRLogger.m

Co-authored-by: Chris Dillard <[email protected]>

---------

Co-authored-by: Chris Dillard <[email protected]>
Co-authored-by: Chris Dillard <[email protected]>
* NR-296618: deallocInteractionHistoryList fix

* fix up dup dealloc Mehtod

* add test interaction history list
@cdillard-NewRelic cdillard-NewRelic marked this pull request as ready for review December 3, 2024 19:27
@cdillard-NewRelic cdillard-NewRelic merged commit 35d204a into staging Dec 3, 2024
16 checks passed
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.

3 participants