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

Cadent Aperture MX: alias cadent_aperture_mx to emx_digital #2912

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

EMXDigital
Copy link
Contributor

No description provided.

@@ -1,6 +1,6 @@
endpoint: "https://hb.emxdgt.com"
maintainer:
email: "[email protected]"
email: "[email protected]"
Copy link
Contributor

Choose a reason for hiding this comment

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

Sent an email to this email address for verification. Please reply "received".

Copy link
Contributor

Choose a reason for hiding this comment

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

Is the contact email address for emx_digital to be changed as well?

Copy link
Contributor Author

@EMXDigital EMXDigital Jul 10, 2023

Choose a reason for hiding this comment

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

We've received and replied to the email address. Both bidder codes/names (cadent_aperture_mx and emx_digital) should be changed to this new @cadent.tv address.

- video
userSync:
iframe:
url: "https://cs.emxdgt.com/um?ssp=pbs&gdpr={{.GDPR}}&gdpr_consent={{.GDPRConsent}}&us_privacy={{.USPrivacy}}&redirect={{.RedirectURL}}"
Copy link
Contributor

Choose a reason for hiding this comment

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

We tried validating this user sync URL but it does not seem to be working. Could you please check? Here is the URL that we are trying -

https://cs.emxdgt.com/um?ssp=pbs&gdpr=&gdpr_consent=&us_privacy=&redirect=https%3A%2F%2Fib.adnxs.com%2F%2Fsetuid%3Fbidder%3Dcadent_aperture_mx%26gdpr%3D%26gdpr_consent%3D%26gpp%3D%26gpp_sid%3D%26f%3Db%26uid%3D%24UID

Copy link
Contributor

Choose a reason for hiding this comment

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

@EMXDigital Requesting to please address this comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gargcreation1992 we are looking into this now, will post an update shortly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gargcreation1992 We've set the endpoint to return a 204 for the time being. Please let us know if this is sufficient for now to get this code merged in.

Copy link
Contributor

Choose a reason for hiding this comment

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

@EMXDigital We would like to validate the user sync URL before this PR is merged.

Copy link
Contributor

Choose a reason for hiding this comment

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

@EMXDigital I am still not able to see UID being set after calling the below endpoint -

https://cs.emxdgt.com/um?ssp=pbs&gdpr=&gdpr_consent=&us_privacy=&redirect=http%3A%2F%2Flocalhost%3A8000%2Fsetuid%3Fbidder%3Dcadent_aperture_mx%26gdpr%3D%26gdpr_consent%3D%26gpp%3D%26gpp_sid%3D%26f%3Db%26uid%3D%24UID

Copy link
Contributor Author

@EMXDigital EMXDigital Jul 28, 2023

Choose a reason for hiding this comment

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

@gargcreation1992 Depending on whether I'm hitting the URL fresh in an incognito browser (i.e., with nothing set on my browser yet), or hitting from an already identified browser, when I just hit that same endpoint in a browser, I am always ultimately 302-redirected to:
https://localhost:8000/setuid?bidder=cadent_aperture_mx&gdpr=&gdpr_consent=&gpp=&gpp_sid=&f=b&uid=[valid value]

Looks like it's filling in the UID [valid value] with an actual real value on my end. What are you seeing happening when you call that endpoint?

Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2023-08-01 at 12 07 09 PM

I am getting 204. Here is the screenshot for your reference and below is the URL that I am trying to hit -

https://cs.emxdgt.com/um?ssp=pbs&gdpr=&gdpr_consent=&us_privacy=&redirect=http%3A%2F%2Flocalhost%3A8000%2Fsetuid%3Fbidder%3Dcadent_aperture_mx%26gdpr%3D%26gdpr_consent%3D%26gpp%3D%26gpp_sid%3D%26f%3Db%26uid%3D%24UID

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gargcreation1992 I see from your profile that you are located in Pune,India. We have geolocation-based 204's. We are only planning on allowing business for now in North America. Are you able to run the same end point through a VPN that is mocking a North America geographic location? You shouldn't be getting a 204 if the Load Balancer thinks you're in North America.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for your patience on this one @EMXDigital! We have tested this on our end and the url is working fine with North American location.

Copy link
Contributor

@gargcreation1992 gargcreation1992 left a comment

Choose a reason for hiding this comment

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

requesting few verifications.

@bsardo bsardo changed the title Cadent Aperture MX Bid Adapter: alias cadent_aperture_mx to emx_digital Cadent Aperture MX: alias cadent_aperture_mx to emx_digital Jul 17, 2023
@gargcreation1992
Copy link
Contributor

@EMXDigital Requesting you to please address the open comment.

@EMXDigital
Copy link
Contributor Author

Resolved merge conflicts.

@EMXDigital EMXDigital reopened this Aug 2, 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.

3 participants