Rafael Taveira

Results 9 comments of Rafael Taveira

@onkarvhanumante Is there any ETA for reviewing this PR? When is the next version expected to be released? TIA

@onkarvhanumante I will review all comments and make the changes accordingly.

@onkarvhanumante you need to use the secure url https:// provided in the nativo.yaml instead of http:// `https://jadserve.postrelease.com/suid/101787?gdpr=&gdpr_consent=&us_privacy=&ntv_gpp_consent=&ntv_r=https%3A%2F%2Fprebid.adnxs.com%2Fpbs%2Fv1%2Fsetuid%3Fbidder%3DNativo%26gdpr%3D%26gdpr_consent%3D%26f%3Di%26uid%3D%22NTV_USER_ID%22`

@bsardo This is the result of the redirect I get: https://prebid.adnxs.com/pbs/v1/setuid?bidder=Nativo&gdpr=&gdpr_consent=&f=i&uid=%22c0ed3c17-4ba0-4d2e-9cf2-cbf8f9037793%22 ![image](https://github.com/user-attachments/assets/56ae952b-b131-449d-b8be-fd087b8b03b4)

> > @bsardo This is the result of the redirect I get: https://prebid.adnxs.com/pbs/v1/setuid?bidder=Nativo&gdpr=&gdpr_consent=&f=i&uid=%22c0ed3c17-4ba0-4d2e-9cf2-cbf8f9037793%22 ![image](https://private-user-images.githubusercontent.com/103446145/371229232-56ae952b-b131-449d-b8be-fd087b8b03b4.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mjc3MDczNzIsIm5iZiI6MTcyNzcwNzA3MiwicGF0aCI6Ii8xMDM0NDYxNDUvMzcxMjI5MjMyLTU2YWU5NTJiLWIxMzEtNDQ5ZC1iOGJlLWZkMDg3YjhiMDNiNC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwOTMwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDkzMFQxNDM3NTJaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT05Yzk0OWFiOWJjNmM3MTc3NjZhZGE3MDU0ZDFhMmI3MjBkOTNjNGIyMDk4MmUwYjY0M2Q3YWZlZTk5OTdiYzhmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.i2wtm2eRiCyGo-cQ2MWXukVyhq4NCLMkIC80Ct9YGG4) > > Hi @rafataveira, I suggest testing your user sync with localhost so that your bidder...

@bsardo I couldn't reproduce this behaviour. As you can see, our server responds with 302 and correctly redirects to the URL defined in the `ntv_r` parameter with the user ID...

@onkarvhanumante @przemkaczmarek @bsardo, is the cookie sync URL the only thing preventing you from approving this PR? We have some clients waiting for the GO version to go live for...

@bsardo I see what is happening to your validation. This is a production cookie sync URL which has different kinds of internal validations to avoid fraud and won't redirect if...

@bsardo we have pushed a fix for the redirect validation. Can you please validate and if all looks good can we merge the PR?