Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Missing conversionEnvelope on iOS for ad conversions #5059

Closed
Uni-verse opened this issue Mar 5, 2022 · 5 comments
Closed

Missing conversionEnvelope on iOS for ad conversions #5059

Uni-verse opened this issue Mar 5, 2022 · 5 comments

Comments

@Uni-verse
Copy link
Contributor

Uni-verse commented Mar 5, 2022

Description:

Missing ConversionEnvelope on iOS for ad conversions

Steps to Reproduce

  1. See Test Plan/Test Steps in Add custom conversion ID patterns for Verifiable Advertiser Conversions brave-core#8640 (comment)

Actual result:
conversionEnvelope not appended to confirmation payload.

https://ads-serve.bravesoftware.com/v2/confirmation/261acb34-dde4-4255-9913-af8ec064e4bd/
{
"blindedPaymentTokens": ["zrO+trqwgPDShe2bG/NRC/mtrZdt8YdMatu2HqjjMBQ="],
"buildChannel": "dev",
"creativeInstanceId": "0e3fa44c-edd4-4ad3-b21e-e6c93ef001c4",
"payload": {},
"platform": "ios",
"publicKey": "JnPs9+7NSdmFflbIvMs0/6BBDt99JMgEvm+9mHNByjI=",
"studies": [],
"type": "conversion"
}

Expected result:
conversionEnvelope is appended to confirmation request

{
"blindedPaymentTokens": ["tiQ/TRMLQhBXkrOAQMV6ydRGQ5P3EcEcd0vCcPC/jEE="],
"buildChannel": "dev",
"conversionEnvelope": {
"alg": "crypto_box_curve25519xsalsa20poly1305",
"ciphertext": "1FZEfhtpqAPx4u3sGHzrwI+7slPT/vt+//1wVE+ZeUdVrLGfE5LjqnM6BvSZkF+z",
"epk": "CQqdjZAKOuMrWDImNqU9Nh8fx0VHNes8rVtsQlzT5i0=",
"nonce": "5rl9XcaibA8hGhGyOBR566FeC8S/XEFI"
},
"creativeInstanceId": "0e3fa44c-edd4-4ad3-b21e-e6c93ef001c4",
"payload": {},
"platform": "ios",
"publicKey": "JnPs9+7NSdmFflbIvMs0/6BBDt99JMgEvm+9mHNByjI=",
"studies": [],
"type": "conversion"
}

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easily

Brave Version:
1.36.108

Device details:
iPhone, iPad
iOS 15, iOS 14

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@Uni-verse Uni-verse changed the title Missing ConversionEnvelope on iOS for ad conversions Missing conversionEnvelope on iOS for ad conversions Mar 5, 2022
@btlechowski
Copy link

btlechowski commented Mar 7, 2022

Steps:

  1. Use catalog in charles: 5059.txt
  2. Clean profile on dev
  3. Switch to staging
  4. Restart browser
  5. Enable rewards and ads
  6. Wait for the notification ad
  7. Convert the ad using: https://www.redbubble.com/check-out/thanks?lol=lol&id=baz-987-zzz
  8. Monitor charles for conversion confirmation`
  9. Make sure conversionEnvelope is attached to the confirmation

Expected result:
conversionEnvelope is attached to the confirmation

Actual result:
conversionEnvelope is not attached to the confirmation

@tmancey
Copy link
Collaborator

tmancey commented Mar 7, 2022

@Uni-verse could you please let me know the brave-core version for the iOS build you were testing, thanks

@Uni-verse
Copy link
Contributor Author

Uni-verse commented Mar 7, 2022

@Uni-verse could you please let me know the brave-core version for the iOS build you were testing, thanks

BraveCore v1.36.108 (99.0.4844.51)

@tmancey

@kjozwiak kjozwiak added this to the 1.36 milestone Mar 8, 2022
@kjozwiak
Copy link
Member

kjozwiak commented Mar 8, 2022

Moved it into the 1.36 milestone and labelled this as release/blocking till we know more and make a decision regarding if this needs to be fixed before we push out 1.36.

@Uni-verse
Copy link
Contributor Author

Uni-verse commented Mar 8, 2022

Verification completed using 1.36 (22.3.8.21) BraveCore 1.36.108 (99.0.4844.51)

PASSED on iPad (7th Gen) running iOS 15.2.1
PASSED on iPhone 12 Pro running iOS 15.1.1
PASSED on iPhone X running iOS 14.4.2

  • Verified conversionEnvelope is appended to the confirmation request
{
	"blindedPaymentTokens": ["tiQ/TRMLQhBXkrOAQMV6ydRGQ5P3EcEcd0vCcPC/jEE="],
	"buildChannel": "dev",
	"conversionEnvelope": {
		"alg": "crypto_box_curve25519xsalsa20poly1305",
		"ciphertext": "1FZEfhtpqAPx4u3sGHzrwI+7slPT/vt+//1wVE+ZeUdVrLGfE5LjqnM6BvSZkF+z",
		"epk": "CQqdjZAKOuMrWDImNqU9Nh8fx0VHNes8rVtsQlzT5i0=",
		"nonce": "5rl9XcaibA8hGhGyOBR566FeC8S/XEFI"
	},
	"creativeInstanceId": "0e3fa44c-edd4-4ad3-b21e-e6c93ef001c4",
	"payload": {},
	"platform": "ios",
	"publicKey": "JnPs9+7NSdmFflbIvMs0/6BBDt99JMgEvm+9mHNByjI=",
	"studies": [],
	"type": "conversion"
}

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

No branches or pull requests

4 participants