Skip to content

Added support for X-EXC-SDK & X-EXC-SDK-Version headers#148

Merged
swarna04 merged 2 commits intoadobe:dev-v4.0.2from
swarna04:MOB-18912
Oct 11, 2023
Merged

Added support for X-EXC-SDK & X-EXC-SDK-Version headers#148
swarna04 merged 2 commits intoadobe:dev-v4.0.2from
swarna04:MOB-18912

Conversation

@swarna04
Copy link
Copy Markdown
Contributor

Description

  • Added support for headers to determine SDK details in Target splunk logs

X-EXC-SDK - AdobeTargetMobile
X-EXC-SDK-Version - <Core+Target SDK version> e.g. 4.0.0+4.0.2

  • Updated Integration tests

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@swarna04 swarna04 changed the base branch from main to dev-v4.0.2 August 10, 2023 19:25
Comment on lines +580 to +582
let sdkVersion = "\(coreVersion)+\(TargetConstants.EXTENSION_VERSION)"
return sdkVersion
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: just return without setting sdkVersion

@swarna04 swarna04 merged commit 68494d2 into adobe:dev-v4.0.2 Oct 11, 2023
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.

2 participants