@ibertran/fortytwo-intra-client 3.0.0 → 3.1.1

package/README.md

@@ -107,9 +107,9 @@ await client.delete('teams/123');
 The `getAll` method automatically handles pagination and returns all results:
 
 ```typescript
-// Get all users from 42 Lyon (handles pagination automatically)
+// Get all users from 42Lyon (handles pagination automatically)
 const lyonUsers = await client.getAll('campus/9/users');
-console.log(`Paris campus has ${lyonUsers.length} users`);
+console.log(`42Lyon campus has ${lyonUsers.length} users`);
 
 // Get all projects with custom page size
 const allProjects = await client.getAll('projects', {

package/dist/index.d.ts

@@ -44,7 +44,13 @@ export declare class FortytwoIntraClient {
     delete(endpoint: URL | string, options?: Omit<inputOptions, "perPage" | "maxPage">): Promise<any>;
     getAll(endpoint: URL | string, options?: Omit<inputOptions, "body">): Promise<any>;
     URL(endpoint: string): URL;
-    getOAuthUrl(redirect_uri?: string | null): string;
+    getOAuthUrl(options?: {
+        redirect_uri?: string;
+        state?: string;
+    }): {
+        url: string;
+        state: string;
+    };
     exchangeOAuthCode(code: string, redirect_uri?: string): Promise<any>;
-    tokenInfos(): Promise<any>;
+    tokenInfos(options: Omit<inputOptions, "body" | "perPage" | "maxPage">): Promise<any>;
 }

package/dist/index.js

@@ -8,6 +8,7 @@ const axios_1 = __importDefault(require("axios"));
 const axios_rate_limit_1 = __importDefault(require("axios-rate-limit"));
 const pagination_1 = require("./lib/pagination");
 const logs_1 = require("./lib/logs");
+const crypto_1 = __importDefault(require("crypto"));
 const defaultConf = {
     redirect_uri: null,
     base_url: "https://api.intra.42.fr/v2/",
@@ -276,17 +277,18 @@ class FortytwoIntraClient {
     URL(endpoint) {
         return new URL(endpoint, this.base_url);
     }
-    getOAuthUrl(redirect_uri = null) {
-        redirect_uri ??= this.redirect_uri;
-        if (!redirect_uri) {
-            throw new Error(`undefined redirect_uri`);
-        }
+    getOAuthUrl(options = {}) {
+        const redirectUri = options.redirect_uri || this.redirect_uri;
+        if (!redirectUri)
+            throw new Error(`Missing redirect_uri parameter`);
+        const state = options.state || crypto_1.default.randomBytes(32).toString('base64url');
         const url = new URL(this.oauth_url);
         url.searchParams.set("client_id", this.client_id);
-        url.searchParams.set("redirect_uri", redirect_uri);
+        url.searchParams.set("redirect_uri", redirectUri);
         url.searchParams.set("response_type", "code");
         url.searchParams.set("scope", this.scopes.join(" "));
-        return url.toString();
+        url.searchParams.set("state", state);
+        return { url: url.toString(), state };
     }
     async exchangeOAuthCode(code, redirect_uri) {
         const res = await this.axiosInstance.post(this.token_url, {
@@ -298,8 +300,8 @@ class FortytwoIntraClient {
         });
         return res?.data;
     }
-    async tokenInfos() {
-        return this.get(this.token_info_url);
+    async tokenInfos(options) {
+        return this.get(this.token_info_url, options);
     }
 }
 exports.FortytwoIntraClient = FortytwoIntraClient;

package/dist/lib/logs.js

@@ -9,12 +9,7 @@ function getLogLine(res, options) {
     const tokens = [];
     // Get query parameters as object
     const queryParams = res.config?.params || {};
-    if (process.stdout.isTTY) {
-        tokens.push(`${green}${res.status}${reset}`);
-    }
-    else {
-        tokens.push(`${res.status}`);
-    }
+    tokens.push(`${green}${res.status}${reset}`);
     tokens.push(options.method.padEnd(6, " "));
     // Add the base URL path
     if (res.config?.url) {
@@ -37,12 +32,7 @@ function getErrorLogLine(err, options) {
     const tokens = [];
     // Get query parameters as object
     const queryParams = err.config?.params || {};
-    if (process.stdout.isTTY) {
-        tokens.push(`${red}${err.status}${reset}`);
-    }
-    else {
-        tokens.push(`${err.status}`);
-    }
+    tokens.push(`${red}${err.status}${reset}`);
     tokens.push(options.method.padEnd(6, " "));
     // Add the base URL path
     if (err.config?.url) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ibertran/fortytwo-intra-client",
-  "version": "3.0.0",
+  "version": "3.1.1",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "scripts": {
@@ -13,7 +13,8 @@
   "devDependencies": {
     "typescript": "^5.9.3",
     "dotenv": "^16.6.1",
-    "ts-node": "^10.9.2"
+    "ts-node": "^10.9.2",
+    "express": "^4.18.2"
   },
   "files": [
     "dist",