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

Enhance AMP debug responses #2406

Open
bretg opened this issue Oct 5, 2022 · 1 comment
Open

Enhance AMP debug responses #2406

bretg opened this issue Oct 5, 2022 · 1 comment

Comments

@bretg
Copy link
Contributor

bretg commented Oct 5, 2022

AMP requests don’t currently have a way to report rejects that happen within the modularity workflow.

When &debug=1 is on the /amp endpoint, right now only ext.debug is supplied on the response. I propose we send the whole seatbid array and other trace info as well – any ORTB that would have been sent to the /auction endpoint should be there in AMP debug mode.

e.g.

{
    "targeting": {
         ...
    },
    "ortb2": {
        ... entire PBS ORTB response here ...
    }
}

If there are automated tests hitting the debug endpoint and parsing the results, then this would be a breaking change because ext.debug would now be ortb2.ext.debug.

@bretg
Copy link
Contributor Author

bretg commented Nov 18, 2022

Discussed in committee. Resolved to move to ortb2 instead of ext and not implement the &verbose=1 flag.

Anyone who's built a monitor using the debug field, will need to update the location where they're looking for debug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Ready for Dev
Development

No branches or pull requests

1 participant