Rep 900

Saying just true/false in the method call was something IntelliJ highlighted saying I should remove the ambiguity by putting it like that.  I just went with it.

Sent from my iPhone

On Oct 4, 2014, at 1:02 PM, Adrian J George III <notifications@github.com<mailto:notifications@github.com>> wrote:


In repose-aggregator/components/filters/openstack-identity-v3/src/main/scala/com/rackspace/papi/components/openstack/identity/v3/OpenStackIdentityV3Handler.scala:

> @@ -121,9 +121,18 @@ class OpenStackIdentityV3Handler(identityConfig: OpenstackIdentityV3Config, iden
>            requestHeaderManager.appendHeader(PowerApiHeader.USER.toString, id, 1.0)
>          }
>          token.get.user.rax_default_region.map { requestHeaderManager.putHeader(OpenStackIdentityV3Headers.X_DEFAULT_REGION.toString, _) }
> -        token.get.project.map { project =>
> -          project.id.map(requestHeaderManager.putHeader(OpenStackIdentityV3Headers.X_PROJECT_ID.toString, _))
> -          project.name.map(requestHeaderManager.putHeader(OpenStackIdentityV3Headers.X_PROJECT_NAME.toString, _))
> +        if(identityConfig.isSendAllProjectIds) {
> +          writeProjectHeader(token.flatMap(_.project.flatMap(_.id)).orNull, token.flatMap(_.roles).orNull, writeAll = true, filterDirector)


writeAll = true?

—
Reply to this email directly or view it on GitHub<https://github.com/rackerlabs/repose/pull/1133/files#r18430731>.