Skip to content

Commit

Permalink
Merge branch '0.8' into 0.9
Browse files Browse the repository at this point in the history
  • Loading branch information
andrerom committed May 13, 2019
2 parents 0b3270c + 0f7532d commit c1ad4af
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/EventSubscriber/UserContextSubscriber.php
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ public function tagUserContext(FilterResponseEvent $event)

// We need to set tag directly on response here to make sure this does not also get applied to the main request
// when using Symfony Proxy, as tag handler does not clear tags between requests.
// OPEN QUESTION: Is SA even loaded for user hash route? If not, using prefix for this won't work.
// IF so change RepositoryTagPrefix to TagPrefixer->prefixTag($tag) or something so we can skip prefix on tags we know need to be global ("all" and "ez-user-context-hash")
// NB: We prefix this even if route is not SA aware, but this is the same as with REST. As doc states,
// it's expected that each repo needs to have own domain so requests against base domain represent same repo.
$response->headers->set($this->tagHeader, $this->prefixService->getRepositoryPrefix() . 'ez-user-context-hash');
}
}

0 comments on commit c1ad4af

Please # to comment.