[profiler] show cookies in separated tab for each request

evaisse · Jul 20, 2015 · 36465c5 · 36465c5
1 parent 5c8edb2
commit 36465c5
Show file tree

Hide file tree

Showing 4 changed files with 95 additions and 4 deletions.
diff --git a/DataCollector/ProfilerDataCollector.php b/DataCollector/ProfilerDataCollector.php
@@ -232,6 +232,7 @@ public function fetchRequestInfos(Request $request)
 
         parse_str($data['queryString'], $data['query']);
         $data['contentType'] = $request->headers->get('content-type');
+        $data['cookies'] = $request->cookies->all();
         return $data;
     }
 
@@ -249,7 +250,22 @@ public function fetchResponseInfos(Response $response)
         $data['headers'] = explode("\r\n\r\n", (string)$response, 2)[0];
         $data['headers'] = explode("\r\n", $data['headers']);
         $data['contentType'] = $response->headers->get('content-type');
-        $data['cookies'] = $response->headers->getCookies();
+        $cookies = $response->headers->getCookies();
+
+        $data['cookies'] = [];
+
+        foreach ($cookies as $c) {
+            $data['cookies'][$c->getName()] = array(
+                "value"     => $c->getValue(),
+                "domain"    => $c->getDomain(),
+                "expires"   => date('Y-m-d H:i:s', $c->getExpiresTime()),
+                "path"      => $c->getPath(),
+                "secure"    => $c->isSecure(),
+                "httpOnly"  => $c->isHttpOnly(),
+                "cleared"   => $c->isCleared(),
+            );
+        }
+
         $data['statusPhrase'] = $data['headers'][0];
 
         return $data;

diff --git a/Resources/views/Collector/partials/call.html.twig b/Resources/views/Collector/partials/call.html.twig
@@ -24,6 +24,9 @@
                 <a href="javascript://">Error</a>
             </li>
             {% endif %}
+            <li>
+                <a href="javascript://">Cookies</a>
+            </li>
             <li>
                 <a href="javascript://">Raw</a>
             </li>
@@ -47,10 +50,12 @@
                     {% include 'SimpleHttpBundle:Collector:partials/error.html.twig' with {'index':index, 'error': call.error} %}
                 </div>
             {% endif %}
-            <div>
+            <div class="tab-content-cookies">
+                {% include 'SimpleHttpBundle:Collector:partials/cookies.html.twig' with {'request': call.request, 'response': call.response} %}
+            </div>
+            <div class="tab-content--raw">
                 {{ dump(call) }}
             </div>
-
         </div>
 
     </div>

diff --git a/Resources/views/Collector/partials/cookies.html.twig b/Resources/views/Collector/partials/cookies.html.twig
@@ -0,0 +1,65 @@
+{% macro display_array_recursive(array, separator = ', ', opening_char = '[', closing_char = ']') -%}
+    {{ opening_char }}
+    {%- for key, value in array -%}
+        {%- if value is iterable -%}
+            {{ key }} => {{ _self.display_array_recursive(value, separator, opening_char, closing_char) }}
+        {%- else -%}
+            {{ key }} => {{ value }}
+            {%- if not loop.last %}{{ separator }}{% endif -%}
+        {%- endif -%}
+    {%- endfor -%}
+    {{ closing_char }}
+{%- endmacro %}
+
+
+<h5>Request Cookies</h5>
+
+
+<table>
+    <tbody>
+    {% for name, parameter in request.cookies %}
+        <tr>
+            <th valign="top">{{ name }}</th>
+            <td>
+                {% if parameter is iterable %}
+                    {{ _self.display_array_recursive(parameter) }}
+                {% else %}
+                    {{ parameter }}
+                {% endif %}
+            </td>
+        </tr>
+    {% endfor %}
+    </tbody>
+</table>
+
+
+{% if response %}
+<h5>Response Cookies</h5>
+
+<table>
+    <tbody>
+    {% for name, parameter in response.cookies %}
+        <tr>
+            {% if parameter.cleared %}
+                <th valign="top" style="text-decoration: line-through">{{ name }}</th>
+            {%  else %}
+                <th valign="top">{{ name }}</th>
+            {% endif %}
+            <td>
+                {% if not parameter.cleared %}
+                    {{ parameter.value }}
+                    <hr style="background: #DDD; border:none; height:1px;" />
+                {% endif %}
+                <div style="color:#888">
+                {% if parameter is iterable %}
+                    {{ _self.display_array_recursive(parameter) }}
+                {% else %}
+                    {{ parameter }}
+                {% endif %}
+                </div>
+            </td>
+        </tr>
+    {% endfor %}
+    </tbody>
+</table>
+{% endif %}
diff --git a/Resources/views/Collector/profiler.html.twig b/Resources/views/Collector/profiler.html.twig
@@ -40,7 +40,12 @@
         <strong>HTTP</strong>
 
         <span class="count">
-            <span>{{ collector.countRequests }}</span>
+            {% if collector.countErrors > 0 %}
+                <span style="background:orange">{{ collector.countRequests }} / {{ collector.countErrors }}</span>
+            {% else %}
+                <span>{{ collector.countRequests }}</span>
+            {% endif %}
+
             {% if (collector.totalTime * 1000) < 1000 %}
                 <span>{{ '%0.0f'|format(collector.totalTime * 1000) }} ms</span>
             {% else %}