Merge pull request #195 from facebook/gFosco.canvasGET-40dev

Removed GET processing from CanvasLoginHelper.

Author: Fosco Marotto

src/Facebook/FacebookCanvasLoginHelper.php

@@ -43,24 +43,12 @@ public function getAppData()
   }
 
   /**
-   * Get raw signed request from either GET or POST.
+   * Get raw signed request from POST.
    *
    * @return string|null
    */
   public function getRawSignedRequest()
   {
-    /**
-     * v2.0 apps use GET for Canvas signed requests.
-     */
-    $rawSignedRequest = $this->getRawSignedRequestFromGet();
-    if ($rawSignedRequest) {
-      return $rawSignedRequest;
-    }
-
-    /**
-     * v1.0 apps use POST for Canvas signed requests, will eventually be
-     * deprecated.
-     */
     $rawSignedRequest = $this->getRawSignedRequestFromPost();
     if ($rawSignedRequest) {
       return $rawSignedRequest;

tests/FacebookCanvasLoginHelperTest.php

@@ -13,15 +13,6 @@ public function setUp()
     $this->helper = new FacebookCanvasLoginHelper('123', 'foo_app_secret');
   }
 
-  public function testSignedRequestDataCanBeRetrievedFromGetData()
-  {
-    $_GET['signed_request'] = $this->rawSignedRequestAuthorized;
-
-    $rawSignedRequest = $this->helper->getRawSignedRequest();
-
-    $this->assertEquals($this->rawSignedRequestAuthorized, $rawSignedRequest);
-  }
-
   public function testSignedRequestDataCanBeRetrievedFromPostData()
   {
     $_POST['signed_request'] = $this->rawSignedRequestAuthorized;