Add hb_env=amp targeting to AMP

[bretg]

I'm working on the Ad Ops documentation and it would be both easier and more consistent if PBS added hb_env=amp to the targeting for AMP responses.

This is similar to what's done now setting hb_env=mobile-app for PBSDK requests

[muuki88]

You can define static key value targetings in a stored request. This is what we do currently

{
  "id" : "xxxxxxxxxxxxxxxxxxx",
  "ext" : {
    "prebid" : {
      "targeting" : { /* ... */ }
      },
      "integration" : "amp",
      "adservertargeting" : [
       // you can add arbitrary targeting here
        {
          "key" : "hb_source",
          "source" : "static",
          "value" : "s2s"
        }
      ]
    }
  },
  "imp" : [ /* ... */  ],
  "source" : { /* ... */  }
}

[bretg]

I would still like to get approval for PBS to add hb_env=amp for the /amp endpoint.

I don't believe this is a breaking change, nor does it need an option like we did for ext.prebid.targeting.includeformat. hb_env=mobile-app is automatically added when app is in the request.

[bretg]

We don't love this, but hb_env already exists. Committee doesn't disapprove. Setting to ready-for-dev.

[bretg]

done with PBS-Java 3.12

[bsardo]

Done in PBS-Go 3.24.0