Bidfluence Adapter update - Single call

[francescocristallo]

Type of change

• Refactoring (no functional changes, no api changes)

Description of change

Same code refactored for sending a single call to the server instead of multiple calls for each bid

• integrations@bidfluence.com
• official adapter submission

[jsnellbaker]

@francescocristallo

While I was testing these changes with the test params in the .md file, I was getting a response back from the server but there was no ad content present.

Below is a copy of the response from the updated endpoint:

{"Bids":[{"BidId":"2dbb3c4341995a","Cpm":0.0,"Width":"300","Height":"250","CreativeId":"","Ad":null}],"UserSyncs":null,"l":""}

When I used the same properties with the previous code/endpoint, I was seeing something return (copied below for reference):

{"BidId":"21bbcddf032d64","Cpm":1E-05,"Width":"300","Height":"250","CreativeId":"","Ad":"<div><a target='_blank' title='Bidfluence' href='http://www.trendworthy.com'><img border='0' src='https://cdn.bidfluence.com/Bdf_300x250.jpg'/></a></div>","UserSyncs":null}

Given the nature of the changes, I wanted to verify the end to end workflow with a test delivery. Do the test params need to be updated to work with the new endpoint? If they should work, can you take a look to see if something is amiss?

[francescocristallo]

@jsnellbaker
The params are the same for both endpoints, I created a test page and used the v1 one and it works. Could you please post the call payload? It could be a security restriction blocking the call our side.

Thanks

[jsnellbaker]

@francescocristallo Below is a copy of the request payload:

{"v":"2.0","azr":true,"ck":true,"re":"http://ap.localhost:9999/integrationExamples/gpt/hello_world.html?pbjs_debug=true","st":["http://ap.localhost:9999/integrationExamples/gpt/hello_world.html?pbjs_debug=true"],"tz":"300","sr":"1680x1050x24","tm":3000,"vp":"1682x348","sdt":"2018/12/05 20:01:47","top":true,"gdpr":false,"gdprc":"","bids":[{"bid":"2e8f27de25bc82","tid":"1000","pid":"1000","rp":0,"w":"300","h":"250"}]}

[francescocristallo]

@jsnellbaker Thanks, as I thought the new endpoint was blocking the localhost address. I fixed it, now it returns the test Ad correctly.

[jsnellbaker]

Thanks for the update; LGTM