Merge pull request #32 from clementmas/master

Define Callback URI
RichieMcMullen · May 14, 2023 · 46b1e01 · 46b1e01
2 parents 2560559 + 2f3f958
commit 46b1e01
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/src/Strava.php b/src/Strava.php
@@ -36,9 +36,17 @@ public function __construct($CLIENT_ID, $CLIENT_SECRET, $REDIRECT_URI, $GUZZLE_C
     #
     # Strava Authenticate
     #
-    public function authenticate($scope='read_all,profile:read_all,activity:read_all')
-    {
-        return redirect('https://www.strava.com/oauth/authorize?client_id='. $this->client_id .'&response_type=code&redirect_uri='. $this->redirect_uri . '&scope=' . $scope . '&state=strava');
+    public function authenticate($scope = 'read_all,profile:read_all,activity:read_all', $redirect_uri = null)
+    {
+        $query = http_build_query([
+            'client_id' => $this->client_id,
+            'response_type' => 'code',
+            'redirect_uri' => $redirect_uri ?: $this->redirect_uri,
+            'scope' => $scope,
+            'state' => 'strava',
+        ]);
+
+        return redirect("https://www.strava.com/oauth/authorize?{$query}");
     }