Brian Sardo

Results 235 comments of Brian Sardo

@ntframeplay are you interested in pushing up a fix?

Hi @sebmil-daily, thanks for your patience. We'll get to this shortly.

PBS-Go has implemented the table in the associated issue #3363 except for the hard alias case. In that scenario `adaptercode` is currently set to the hard alias just like the...

Hi @rafataveira, your tests are failing because you need to run `go fmt`.

> works: ![image](https://private-user-images.githubusercontent.com/167743744/370715081-9a577a27-5643-4152-b271-e69b8590bf81.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjcyNzY3MjcsIm5iZiI6MTcyNzI3NjQyNywicGF0aCI6Ii8xNjc3NDM3NDQvMzcwNzE1MDgxLTlhNTc3YTI3LTU2NDMtNDE1Mi1iMjcxLWU2OWI4NTkwYmY4MS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwOTI1JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDkyNVQxNTAwMjdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1hZjljYzc2NDU2ZDJjZmNiYTI3YWVkNTczMTgwYTRkMzZjODBlMzk5ZTJlYzc0NGM4ZTBiZjAzMjUxMDc1ZmFkJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.vXwVLUVTlpuLRKPgLknqk7eG6IICv-lamIgo9aen59A) I see this 200 response as well but it is supposed to redirect (302) to the PBS setuid endpoint which is specified in your `ntv_r` parameter. @rafataveira

> @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 is known and...

@rafataveira we're still seeing an issue with the user sync. Your server is responding with a 200. Since you've classified this user sync as type redirect, your server should be...

> @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...

@rafataveira we're required to verify the user sync endpoints works. Can you either provide details on how to construct a request so that it is not flagged by your servers...

Hi @aparekh-nativo, @rafataveira tests are failing and your PR branch does not run because you have some incorrect package references after merging with master. All package imports should reference v3...