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

Aliases AppnexusAstBidder #1682

Closed
Deimos01 opened this issue Oct 11, 2017 · 5 comments
Closed

Aliases AppnexusAstBidder #1682

Deimos01 opened this issue Oct 11, 2017 · 5 comments
Assignees

Comments

@Deimos01
Copy link

Type of issue

Bug/Question

Description

I'm trying to use Brealtime with the AppnexusAst bidder. Is it still possible ?

Steps to reproduce

I'm trying to implement aliases with the new specs : http://prebid.org/dev-docs/bidder-adapter-1.html
Just added this line on the AppnexusAstBidAdapter.js

export const spec = {
  code: BIDDER_CODE,
  aliases :['brealtime'],
  supportedMediaTypes: [VIDEO, NATIVE],

Test page

https://www.themoneytizer.com/demo/geoprebid?pbjs_debug=true

Actual results

image

Platform details

Prebid version 0.30.1

Thank you !

@jaiminpanchal27
Copy link
Collaborator

Yes it is possible. Whenever something is wrong with response bidderFactory handles that and logs a generic error message as you see that above.

You can see the error response in network tab.

For AppnexusAst, I have created #1697 to log actual error message received from endpoint to find root cause easily.

@jaiminpanchal27
Copy link
Collaborator

#1697 is merged so closing this.

@Deimos01
Copy link
Author

Hello @jaiminpanchal27 ,

So i have to wait the next Prebid release to use Brealtime with the AppnexusAst bidder ?

Thank you very much.

@jaiminpanchal27
Copy link
Collaborator

jaiminpanchal27 commented Oct 17, 2017

@Deimos01 Its not a blocker for you. This is the response for brealTime Request {"error":"Could not find valid member in request"} on your test page. I think you need to update your config.

In #1697 i am logging that error message in console to help debug faster.

Also we are doing a release today so you will see the error message logged in console

@Deimos01
Copy link
Author

Thank you very much @jaiminpanchal27 !

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

No branches or pull requests

2 participants