1 | <?php |
||
27 | class UserContextSubscriber implements EventSubscriberInterface |
||
28 | { |
||
29 | /** |
||
30 | * The options configured in the constructor argument or default values. |
||
31 | * |
||
32 | * @var array |
||
33 | */ |
||
34 | private $options; |
||
35 | |||
36 | /** |
||
37 | * Generated user hash. |
||
38 | * |
||
39 | * @var string |
||
40 | */ |
||
41 | private $userHash; |
||
42 | |||
43 | /** |
||
44 | * When creating this subscriber, you can configure a number of options. |
||
45 | * |
||
46 | * - anonymous_hash: Hash used for anonymous user. |
||
47 | * - user_hash_accept_header: Accept header value to be used to request the user hash to the |
||
48 | * backend application. Must match the setup of the backend application. |
||
49 | * - user_hash_header: Name of the header the user context hash will be stored into. Must |
||
50 | * match the setup for the Vary header in the backend application. |
||
51 | * - user_hash_uri: Target URI used in the request for user context hash generation. |
||
52 | * - user_hash_method: HTTP Method used with the hash lookup request for user context hash generation. |
||
53 | * - session_name_prefix: Prefix for session cookies. Must match your PHP session configuration. |
||
54 | * |
||
55 | * @param array $options Options to overwrite the default options |
||
56 | * |
||
57 | * @throws \InvalidArgumentException if unknown keys are found in $options |
||
58 | */ |
||
59 | 12 | public function __construct(array $options = []) |
|
60 | { |
||
61 | 12 | $resolver = new OptionsResolver(); |
|
62 | 12 | $resolver->setDefaults([ |
|
63 | 12 | 'anonymous_hash' => '38015b703d82206ebc01d17a39c727e5', |
|
64 | 'user_hash_accept_header' => 'application/vnd.fos.user-context-hash', |
||
65 | 'user_hash_header' => 'X-User-Context-Hash', |
||
66 | 'user_hash_uri' => '/_fos_user_context_hash', |
||
67 | 'user_hash_method' => 'GET', |
||
68 | 'session_name_prefix' => 'PHPSESSID', |
||
69 | ]); |
||
70 | |||
71 | 12 | $this->options = $resolver->resolve($options); |
|
72 | 11 | } |
|
73 | |||
74 | /** |
||
75 | * {@inheritdoc} |
||
76 | */ |
||
77 | 1 | public static function getSubscribedEvents() |
|
78 | { |
||
79 | return [ |
||
80 | 1 | Events::PRE_HANDLE => 'preHandle', |
|
81 | ]; |
||
82 | } |
||
83 | |||
84 | /** |
||
85 | * Look at the request before it is handled by the kernel. |
||
86 | * |
||
87 | * Adds the user hash header to the request. |
||
88 | * |
||
89 | * Checks if an external request tries tampering with the use context hash mechanism |
||
90 | * to prevent attacks. |
||
91 | * |
||
92 | * @param CacheEvent $event |
||
93 | */ |
||
94 | 11 | public function preHandle(CacheEvent $event) |
|
95 | { |
||
96 | 11 | $request = $event->getRequest(); |
|
97 | 11 | if (!$this->isInternalRequest($request)) { |
|
98 | // Prevent tampering attacks on the hash mechanism |
||
99 | 11 | if ($request->headers->get('accept') === $this->options['user_hash_accept_header'] |
|
100 | 11 | || $request->headers->get($this->options['user_hash_header']) !== null |
|
101 | ) { |
||
102 | 4 | $event->setResponse(new Response('', 400)); |
|
103 | |||
104 | 4 | return; |
|
105 | } |
||
106 | |||
107 | 7 | if ($request->isMethodSafe()) { |
|
108 | 7 | $request->headers->set($this->options['user_hash_header'], $this->getUserHash($event->getKernel(), $request)); |
|
109 | } |
||
110 | } |
||
111 | |||
112 | // let the kernel handle this request. |
||
113 | 7 | } |
|
114 | |||
115 | /** |
||
116 | * Remove unneeded things from the request for user hash generation. |
||
117 | * |
||
118 | * Cleans cookies header to only keep the session identifier cookie, so the hash lookup request |
||
119 | * can be cached per session. |
||
120 | * |
||
121 | * @param Request $hashLookupRequest |
||
122 | * @param Request $originalRequest |
||
123 | */ |
||
124 | 4 | protected function cleanupHashLookupRequest(Request $hashLookupRequest, Request $originalRequest) |
|
125 | { |
||
126 | 4 | $sessionIds = []; |
|
127 | 4 | foreach ($originalRequest->cookies as $name => $value) { |
|
128 | 4 | if ($this->isSessionName($name)) { |
|
129 | 2 | $sessionIds[$name] = $value; |
|
130 | 4 | $hashLookupRequest->cookies->set($name, $value); |
|
131 | } |
||
132 | } |
||
133 | |||
134 | 4 | if (count($sessionIds) > 0) { |
|
135 | 2 | $hashLookupRequest->headers->set('Cookie', http_build_query($sessionIds, '', '; ')); |
|
136 | } |
||
137 | 4 | } |
|
138 | |||
139 | /** |
||
140 | * Checks if passed request object is to be considered internal (e.g. for user hash lookup). |
||
141 | * |
||
142 | * @param Request $request |
||
143 | * |
||
144 | * @return bool |
||
145 | */ |
||
146 | 11 | private function isInternalRequest(Request $request) |
|
150 | |||
151 | /** |
||
152 | * Returns the user context hash for $request. |
||
153 | * |
||
154 | * @param HttpKernelInterface $kernel |
||
155 | * @param Request $request |
||
156 | * |
||
157 | * @return string |
||
158 | */ |
||
159 | 7 | private function getUserHash(HttpKernelInterface $kernel, Request $request) |
|
177 | |||
178 | /** |
||
179 | * Checks if current request is considered anonymous. |
||
180 | * |
||
181 | * @param Request $request |
||
182 | * |
||
183 | * @return bool |
||
184 | */ |
||
185 | 7 | private function isAnonymous(Request $request) |
|
186 | { |
||
187 | // You might have to enable rewriting of the Authorization header in your server config or .htaccess: |
||
188 | // RewriteEngine On |
||
189 | // RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}] |
||
190 | 7 | if ($request->server->has('AUTHORIZATION') || |
|
191 | 7 | $request->server->has('HTTP_AUTHORIZATION') || |
|
192 | 7 | $request->server->has('PHP_AUTH_USER') |
|
193 | ) { |
||
194 | 2 | return false; |
|
195 | } |
||
196 | |||
197 | 5 | foreach ($request->cookies as $name => $value) { |
|
198 | 2 | if ($this->isSessionName($name)) { |
|
199 | 2 | return false; |
|
200 | } |
||
201 | } |
||
202 | |||
203 | 3 | return true; |
|
204 | } |
||
205 | |||
206 | /** |
||
207 | * Checks if passed string can be considered as a session name, such as would be used in cookies. |
||
208 | * |
||
209 | * @param string $name |
||
210 | * |
||
211 | * @return bool |
||
212 | */ |
||
213 | 4 | private function isSessionName($name) |
|
217 | |||
218 | /** |
||
219 | * Generates the request object that will be forwarded to get the user context hash. |
||
220 | * |
||
221 | * @param Request $request |
||
222 | * |
||
223 | * @return Request The request that will return the user context hash value |
||
224 | */ |
||
225 | 4 | private function generateHashLookupRequest(Request $request) |
|
234 | } |
||
235 |
Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly.
For example, imagine you have a variable
$accountId
that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to theid
property of an instance of theAccount
class. This class holds a proper account, so the id value must no longer be false.Either this assignment is in error or a type check should be added for that assignment.