Passed
Pull Request — master (#11)
by Tim
01:42
created
lib/Cas/Ticket/SQLTicketStore.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
         $dsn = $storeConfig->getString('dsn');
53 53
         $username = $storeConfig->getString('username');
54 54
         $password = $storeConfig->getString('password');
55
-        $options =  $storeConfig->getArray('options', []);
55
+        $options = $storeConfig->getArray('options', []);
56 56
         $this->prefix = $storeConfig->getString('prefix', '');
57 57
 
58 58
         $this->pdo = new \PDO($dsn, $username, $password, $options);
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
         }
133 133
 
134 134
         while (($row = $fetchTableVersion->fetch(\PDO::FETCH_ASSOC)) !== false) {
135
-            $this->tableVersions[$row['_name']] = (int)$row['_version'];
135
+            $this->tableVersions[$row['_name']] = (int) $row['_version'];
136 136
         }
137 137
     }
138 138
 
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
             $insertQuery->execute($data);
231 231
             return;
232 232
         } catch (\PDOException $e) {
233
-            $ecode = (string)$e->getCode();
233
+            $ecode = (string) $e->getCode();
234 234
             switch ($ecode) {
235 235
                 case '23505': /* PostgreSQL */
236 236
                     break;
Please login to merge, or discard this patch.
www/utility/validateTicket.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -82,8 +82,8 @@  discard block
 block discarded – undo
82 82
                             'sessionId' => $serviceTicket['sessionId']
83 83
                         ]);
84 84
                         try {
85
-                            \SimpleSAML\Utils\HTTP::fetch($pgtUrl . '?pgtIou=' . $proxyGrantingTicket['iou']
86
-                                . '&pgtId=' . $proxyGrantingTicket['id']);
85
+                            \SimpleSAML\Utils\HTTP::fetch($pgtUrl.'?pgtIou='.$proxyGrantingTicket['iou']
86
+                                . '&pgtId='.$proxyGrantingTicket['id']);
87 87
 
88 88
                             $protocol->setProxyGrantingTicketIOU($proxyGrantingTicket['iou']);
89 89
 
@@ -96,29 +96,29 @@  discard block
 block discarded – undo
96 96
                 echo $protocol->getValidateSuccessResponse($serviceTicket['userName']);
97 97
             } else {
98 98
                 if ($ticketFactory->isExpired($serviceTicket)) {
99
-                    $message = 'Ticket ' . var_export($_GET['ticket'], true) . ' has expired';
99
+                    $message = 'Ticket '.var_export($_GET['ticket'], true).' has expired';
100 100
 
101
-                    \SimpleSAML\Logger::debug('casserver:' . $message);
101
+                    \SimpleSAML\Logger::debug('casserver:'.$message);
102 102
 
103 103
                     echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
104 104
                 } else {
105 105
                     if (sanitize($serviceTicket['service']) != sanitize($_GET['service'])) {
106 106
                         $message = 'Mismatching service parameters: expected '
107 107
                             . var_export($serviceTicket['service'], true)
108
-                            . ' but was: ' . var_export($_GET['service'], true);
108
+                            . ' but was: '.var_export($_GET['service'], true);
109 109
 
110
-                        \SimpleSAML\Logger::debug('casserver:' . $message);
110
+                        \SimpleSAML\Logger::debug('casserver:'.$message);
111 111
 
112 112
                         echo $protocol->getValidateFailureResponse('INVALID_SERVICE', $message);
113 113
                     } else {
114 114
                         if ($serviceTicket['forceAuthn'] != $forceAuthn) {
115 115
                             $message = 'Ticket was issue from single sign on session';
116 116
 
117
-                            \SimpleSAML\Logger::debug('casserver:' . $message);
117
+                            \SimpleSAML\Logger::debug('casserver:'.$message);
118 118
 
119 119
                             echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
120 120
                         } else {
121
-                            \SimpleSAML\Logger::error('casserver:' . $method . ': internal server error.');
121
+                            \SimpleSAML\Logger::error('casserver:'.$method.': internal server error.');
122 122
 
123 123
                             echo $protocol->getValidateFailureResponse('INTERNAL_ERROR', 'Unknown internal error');
124 124
                         }
@@ -127,23 +127,23 @@  discard block
 block discarded – undo
127 127
             }
128 128
         } else {
129 129
             if (is_null($serviceTicket)) {
130
-                $message = 'Ticket ' . var_export($_GET['ticket'], true) . ' not recognized';
130
+                $message = 'Ticket '.var_export($_GET['ticket'], true).' not recognized';
131 131
 
132
-                \SimpleSAML\Logger::debug('casserver:' . $message);
132
+                \SimpleSAML\Logger::debug('casserver:'.$message);
133 133
 
134 134
                 echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
135 135
             } else {
136 136
                 if ($ticketFactory->isProxyTicket($serviceTicket) && ($method === 'serviceValidate')) {
137
-                    $message = 'Ticket ' . var_export($_GET['ticket'], true)
137
+                    $message = 'Ticket '.var_export($_GET['ticket'], true)
138 138
                         . ' is a proxy ticket. Use proxyValidate instead.';
139 139
 
140
-                    \SimpleSAML\Logger::debug('casserver:' . $message);
140
+                    \SimpleSAML\Logger::debug('casserver:'.$message);
141 141
 
142 142
                     echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
143 143
                 } else {
144
-                    $message = 'Ticket ' . var_export($_GET['ticket'], true) . ' is not a service ticket';
144
+                    $message = 'Ticket '.var_export($_GET['ticket'], true).' is not a service ticket';
145 145
 
146
-                    \SimpleSAML\Logger::debug('casserver:' . $message);
146
+                    \SimpleSAML\Logger::debug('casserver:'.$message);
147 147
 
148 148
                     echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
149 149
                 }
@@ -160,13 +160,13 @@  discard block
 block discarded – undo
160 160
     if (!array_key_exists('service', $_GET)) {
161 161
         $message = 'Missing service parameter: [service]';
162 162
 
163
-        \SimpleSAML\Logger::debug('casserver:' . $message);
163
+        \SimpleSAML\Logger::debug('casserver:'.$message);
164 164
 
165 165
         echo $protocol->getValidateFailureResponse('INVALID_REQUEST', $message);
166 166
     } else {
167 167
         $message = 'Missing ticket parameter: [ticket]';
168 168
 
169
-        \SimpleSAML\Logger::debug('casserver:' . $message);
169
+        \SimpleSAML\Logger::debug('casserver:'.$message);
170 170
 
171 171
         echo $protocol->getValidateFailureResponse('INVALID_REQUEST', $message);
172 172
     }
Please login to merge, or discard this patch.