Skip to content

Fix spurious check for macro in MoPub ExchangeConnector - #18

Open
jsulmont wants to merge 2 commits into
masterfrom
mopub_fix
Open

Fix spurious check for macro in MoPub ExchangeConnector#18
jsulmont wants to merge 2 commits into
masterfrom
mopub_fix

Conversation

@jsulmont

Copy link
Copy Markdown

Please merge ASAP --

@mikebd

mikebd commented Apr 17, 2014

Copy link
Copy Markdown

MoPub does not support encryption, so it is not expecting ${AUCTION_PRICE:BF}.

We should check for ${AUCTION_PRICE} in the win notification nurl. That is required to pass the MoPub sandbox integration testing. ${AUCTION_PRICE} may also be provided in adm but MoPub considers nurl as authoritative and will not be satisfied by the presence of ${AUCTION_PRICE} in adm.

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.

2 participants