Merge pull request #156 from getmaxun/jwt-fix
fix: gracefully handle jwt token decoding
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
import { Model, DataTypes, Optional } from 'sequelize';
|
||||
import sequelize from '../storage/db';
|
||||
import { WorkflowFile, Where, What, WhereWhatPair } from 'maxun-core';
|
||||
import { Model, DataTypes, Optional } from "sequelize";
|
||||
import sequelize from "../storage/db";
|
||||
import { WorkflowFile, Where, What, WhereWhatPair } from "maxun-core";
|
||||
|
||||
interface RobotMeta {
|
||||
name: string;
|
||||
@@ -15,23 +15,42 @@ interface RobotWorkflow {
|
||||
workflow: WhereWhatPair[];
|
||||
}
|
||||
|
||||
interface IntegrationData {
|
||||
google_sheets?: {
|
||||
email: string;
|
||||
sheet_id: string;
|
||||
sheet_name: string;
|
||||
access_token: string;
|
||||
refresh_token: string;
|
||||
};
|
||||
airtable?: {
|
||||
base_id: string;
|
||||
table_name: string;
|
||||
access_token: string;
|
||||
refresh_token: string;
|
||||
};
|
||||
}
|
||||
|
||||
interface RobotAttributes {
|
||||
id: string;
|
||||
userId?: number;
|
||||
recording_meta: RobotMeta;
|
||||
recording: RobotWorkflow;
|
||||
google_sheet_email?: string | null;
|
||||
google_sheet_name?: string | null;
|
||||
google_sheet_id?: string | null;
|
||||
google_access_token?: string | null;
|
||||
google_refresh_token?: string | null;
|
||||
schedule?: ScheduleConfig | null;
|
||||
integrations?: IntegrationData | null;
|
||||
}
|
||||
|
||||
interface ScheduleConfig {
|
||||
runEvery: number;
|
||||
runEveryUnit: 'MINUTES' | 'HOURS' | 'DAYS' | 'WEEKS' | 'MONTHS';
|
||||
startFrom: 'SUNDAY' | 'MONDAY' | 'TUESDAY' | 'WEDNESDAY' | 'THURSDAY' | 'FRIDAY' | 'SATURDAY';
|
||||
runEveryUnit: "MINUTES" | "HOURS" | "DAYS" | "WEEKS" | "MONTHS";
|
||||
startFrom:
|
||||
| "SUNDAY"
|
||||
| "MONDAY"
|
||||
| "TUESDAY"
|
||||
| "WEDNESDAY"
|
||||
| "THURSDAY"
|
||||
| "FRIDAY"
|
||||
| "SATURDAY";
|
||||
atTimeStart?: string;
|
||||
atTimeEnd?: string;
|
||||
timezone: string;
|
||||
@@ -41,19 +60,18 @@ interface ScheduleConfig {
|
||||
cronExpression?: string;
|
||||
}
|
||||
|
||||
interface RobotCreationAttributes extends Optional<RobotAttributes, 'id'> { }
|
||||
interface RobotCreationAttributes extends Optional<RobotAttributes, "id"> {}
|
||||
|
||||
class Robot extends Model<RobotAttributes, RobotCreationAttributes> implements RobotAttributes {
|
||||
class Robot
|
||||
extends Model<RobotAttributes, RobotCreationAttributes>
|
||||
implements RobotAttributes
|
||||
{
|
||||
public id!: string;
|
||||
public userId!: number;
|
||||
public recording_meta!: RobotMeta;
|
||||
public recording!: RobotWorkflow;
|
||||
public google_sheet_email!: string | null;
|
||||
public google_sheet_name?: string | null;
|
||||
public google_sheet_id?: string | null;
|
||||
public google_access_token!: string | null;
|
||||
public google_refresh_token!: string | null;
|
||||
public schedule!: ScheduleConfig | null;
|
||||
public integrations!: IntegrationData | null;
|
||||
}
|
||||
|
||||
Robot.init(
|
||||
@@ -75,25 +93,10 @@ Robot.init(
|
||||
type: DataTypes.JSONB,
|
||||
allowNull: false,
|
||||
},
|
||||
google_sheet_email: {
|
||||
type: DataTypes.STRING,
|
||||
allowNull: true,
|
||||
},
|
||||
google_sheet_name: {
|
||||
type: DataTypes.STRING,
|
||||
allowNull: true,
|
||||
},
|
||||
google_sheet_id: {
|
||||
type: DataTypes.STRING,
|
||||
allowNull: true,
|
||||
},
|
||||
google_access_token: {
|
||||
type: DataTypes.STRING,
|
||||
allowNull: true,
|
||||
},
|
||||
google_refresh_token: {
|
||||
type: DataTypes.STRING,
|
||||
integrations: {
|
||||
type: DataTypes.JSONB,
|
||||
allowNull: true,
|
||||
defaultValue: {},
|
||||
},
|
||||
schedule: {
|
||||
type: DataTypes.JSONB,
|
||||
@@ -102,7 +105,7 @@ Robot.init(
|
||||
},
|
||||
{
|
||||
sequelize,
|
||||
tableName: 'robot',
|
||||
tableName: "robot",
|
||||
timestamps: false,
|
||||
}
|
||||
);
|
||||
@@ -112,4 +115,4 @@ Robot.init(
|
||||
// as: 'runs', // Alias for the relation
|
||||
// });
|
||||
|
||||
export default Robot;
|
||||
export default Robot;
|
||||
|
||||
@@ -6,13 +6,14 @@ import Robot from "../../models/Robot";
|
||||
interface GoogleSheetUpdateTask {
|
||||
robotId: string;
|
||||
runId: string;
|
||||
status: 'pending' | 'completed' | 'failed';
|
||||
status: "pending" | "completed" | "failed";
|
||||
retries: number;
|
||||
}
|
||||
|
||||
const MAX_RETRIES = 5;
|
||||
|
||||
export let googleSheetUpdateTasks: { [runId: string]: GoogleSheetUpdateTask } = {};
|
||||
export let googleSheetUpdateTasks: { [runId: string]: GoogleSheetUpdateTask } =
|
||||
{};
|
||||
|
||||
export async function updateGoogleSheet(robotId: string, runId: string) {
|
||||
try {
|
||||
@@ -24,51 +25,74 @@ export async function updateGoogleSheet(robotId: string, runId: string) {
|
||||
|
||||
const plainRun = run.toJSON();
|
||||
|
||||
if (plainRun.status === 'success') {
|
||||
if (plainRun.status === "success") {
|
||||
let data: { [key: string]: any }[] = [];
|
||||
if (plainRun.serializableOutput && Object.keys(plainRun.serializableOutput).length > 0) {
|
||||
data = plainRun.serializableOutput['item-0'] as { [key: string]: any }[];
|
||||
|
||||
} else if (plainRun.binaryOutput && plainRun.binaryOutput['item-0']) {
|
||||
if (
|
||||
plainRun.serializableOutput &&
|
||||
Object.keys(plainRun.serializableOutput).length > 0
|
||||
) {
|
||||
data = plainRun.serializableOutput["item-0"] as {
|
||||
[key: string]: any;
|
||||
}[];
|
||||
} else if (plainRun.binaryOutput && plainRun.binaryOutput["item-0"]) {
|
||||
// Handle binaryOutput by setting the URL as a data entry
|
||||
const binaryUrl = plainRun.binaryOutput['item-0'] as string;
|
||||
|
||||
const binaryUrl = plainRun.binaryOutput["item-0"] as string;
|
||||
|
||||
// Create a placeholder object with the binary URL
|
||||
data = [{ "Screenshot URL": binaryUrl }];
|
||||
}
|
||||
|
||||
const robot = await Robot.findOne({ where: { 'recording_meta.id': robotId } });
|
||||
const robot = await Robot.findOne({
|
||||
where: { "recording_meta.id": robotId },
|
||||
});
|
||||
|
||||
if (!robot) {
|
||||
throw new Error(`Robot not found for robotId: ${robotId}`);
|
||||
}
|
||||
|
||||
const plainRobot = robot.toJSON();
|
||||
|
||||
const spreadsheetId = plainRobot.integrations?.google_sheets?.sheet_id;
|
||||
|
||||
const spreadsheetId = plainRobot.google_sheet_id;
|
||||
if (plainRobot.google_sheet_email && spreadsheetId) {
|
||||
console.log(`Preparing to write data to Google Sheet for robot: ${robotId}, spreadsheetId: ${spreadsheetId}`);
|
||||
if (plainRobot.integrations?.google_sheets?.email && spreadsheetId) {
|
||||
console.log(
|
||||
`Preparing to write data to Google Sheet for robot: ${robotId}, spreadsheetId: ${spreadsheetId}`
|
||||
);
|
||||
|
||||
const headers = Object.keys(data[0]);
|
||||
const rows = data.map((row: { [key: string]: any }) => Object.values(row));
|
||||
const rows = data.map((row: { [key: string]: any }) =>
|
||||
Object.values(row)
|
||||
);
|
||||
const outputData = [headers, ...rows];
|
||||
|
||||
await writeDataToSheet(robotId, spreadsheetId, outputData);
|
||||
console.log(`Data written to Google Sheet successfully for Robot: ${robotId} and Run: ${runId}`);
|
||||
console.log(
|
||||
`Data written to Google Sheet successfully for Robot: ${robotId} and Run: ${runId}`
|
||||
);
|
||||
} else {
|
||||
console.log('Google Sheets integration not configured.');
|
||||
console.log("Google Sheets integration not configured.");
|
||||
}
|
||||
} else {
|
||||
console.log('Run status is not success or serializableOutput is missing.');
|
||||
console.log(
|
||||
"Run status is not success or serializableOutput is missing."
|
||||
);
|
||||
}
|
||||
} catch (error: any) {
|
||||
console.error(`Failed to write data to Google Sheet for Robot: ${robotId} and Run: ${runId}: ${error.message}`);
|
||||
console.error(
|
||||
`Failed to write data to Google Sheet for Robot: ${robotId} and Run: ${runId}: ${error.message}`
|
||||
);
|
||||
}
|
||||
};
|
||||
}
|
||||
|
||||
export async function writeDataToSheet(robotId: string, spreadsheetId: string, data: any[]) {
|
||||
export async function writeDataToSheet(
|
||||
robotId: string,
|
||||
spreadsheetId: string,
|
||||
data: any[]
|
||||
) {
|
||||
try {
|
||||
const robot = await Robot.findOne({ where: { 'recording_meta.id': robotId } });
|
||||
const robot = await Robot.findOne({
|
||||
where: { "recording_meta.id": robotId },
|
||||
});
|
||||
|
||||
if (!robot) {
|
||||
throw new Error(`Robot not found for robotId: ${robotId}`);
|
||||
@@ -76,8 +100,11 @@ export async function writeDataToSheet(robotId: string, spreadsheetId: string, d
|
||||
|
||||
const plainRobot = robot.toJSON();
|
||||
|
||||
if (!plainRobot.google_access_token || !plainRobot.google_refresh_token) {
|
||||
throw new Error('Google Sheets access not configured for user');
|
||||
const access_token = plainRobot.integrations?.google_sheets?.access_token;
|
||||
const refresh_token = plainRobot.integrations?.google_sheets?.refresh_token;
|
||||
|
||||
if (!access_token || !refresh_token) {
|
||||
throw new Error("Google Sheets access not configured for user");
|
||||
}
|
||||
|
||||
const oauth2Client = new google.auth.OAuth2(
|
||||
@@ -87,11 +114,11 @@ export async function writeDataToSheet(robotId: string, spreadsheetId: string, d
|
||||
);
|
||||
|
||||
oauth2Client.setCredentials({
|
||||
access_token: plainRobot.google_access_token,
|
||||
refresh_token: plainRobot.google_refresh_token,
|
||||
access_token: access_token,
|
||||
refresh_token: refresh_token,
|
||||
});
|
||||
|
||||
oauth2Client.on('tokens', async (tokens) => {
|
||||
oauth2Client.on("tokens", async (tokens) => {
|
||||
if (tokens.refresh_token) {
|
||||
await robot.update({ google_refresh_token: tokens.refresh_token });
|
||||
}
|
||||
@@ -100,22 +127,22 @@ export async function writeDataToSheet(robotId: string, spreadsheetId: string, d
|
||||
}
|
||||
});
|
||||
|
||||
const sheets = google.sheets({ version: 'v4', auth: oauth2Client });
|
||||
const sheets = google.sheets({ version: "v4", auth: oauth2Client });
|
||||
|
||||
const resource = { values: data };
|
||||
console.log('Attempting to write to spreadsheet:', spreadsheetId);
|
||||
console.log("Attempting to write to spreadsheet:", spreadsheetId);
|
||||
|
||||
const response = await sheets.spreadsheets.values.append({
|
||||
spreadsheetId,
|
||||
range: 'Sheet1!A1',
|
||||
valueInputOption: 'USER_ENTERED',
|
||||
range: "Sheet1!A1",
|
||||
valueInputOption: "USER_ENTERED",
|
||||
requestBody: resource,
|
||||
});
|
||||
|
||||
if (response.status === 200) {
|
||||
console.log('Data successfully appended to Google Sheet.');
|
||||
console.log("Data successfully appended to Google Sheet.");
|
||||
} else {
|
||||
console.error('Google Sheets append failed:', response);
|
||||
console.error("Google Sheets append failed:", response);
|
||||
}
|
||||
|
||||
logger.log(`info`, `Data written to Google Sheet: ${spreadsheetId}`);
|
||||
@@ -130,33 +157,42 @@ export const processGoogleSheetUpdates = async () => {
|
||||
let hasPendingTasks = false;
|
||||
for (const runId in googleSheetUpdateTasks) {
|
||||
const task = googleSheetUpdateTasks[runId];
|
||||
console.log(`Processing task for runId: ${runId}, status: ${task.status}`);
|
||||
console.log(
|
||||
`Processing task for runId: ${runId}, status: ${task.status}`
|
||||
);
|
||||
|
||||
if (task.status === 'pending') {
|
||||
if (task.status === "pending") {
|
||||
hasPendingTasks = true;
|
||||
try {
|
||||
await updateGoogleSheet(task.robotId, task.runId);
|
||||
console.log(`Successfully updated Google Sheet for runId: ${runId}`);
|
||||
delete googleSheetUpdateTasks[runId];
|
||||
} catch (error: any) {
|
||||
console.error(`Failed to update Google Sheets for run ${task.runId}:`, error);
|
||||
console.error(
|
||||
`Failed to update Google Sheets for run ${task.runId}:`,
|
||||
error
|
||||
);
|
||||
if (task.retries < MAX_RETRIES) {
|
||||
googleSheetUpdateTasks[runId].retries += 1;
|
||||
console.log(`Retrying task for runId: ${runId}, attempt: ${task.retries}`);
|
||||
console.log(
|
||||
`Retrying task for runId: ${runId}, attempt: ${task.retries}`
|
||||
);
|
||||
} else {
|
||||
googleSheetUpdateTasks[runId].status = 'failed';
|
||||
console.log(`Max retries reached for runId: ${runId}. Marking task as failed.`);
|
||||
googleSheetUpdateTasks[runId].status = "failed";
|
||||
console.log(
|
||||
`Max retries reached for runId: ${runId}. Marking task as failed.`
|
||||
);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (!hasPendingTasks) {
|
||||
console.log('No pending tasks. Exiting loop.');
|
||||
console.log("No pending tasks. Exiting loop.");
|
||||
break;
|
||||
}
|
||||
|
||||
console.log('Waiting for 5 seconds before checking again...');
|
||||
await new Promise(resolve => setTimeout(resolve, 5000));
|
||||
console.log("Waiting for 5 seconds before checking again...");
|
||||
await new Promise((resolve) => setTimeout(resolve, 5000));
|
||||
}
|
||||
};
|
||||
};
|
||||
|
||||
@@ -60,18 +60,28 @@ const AuthProvider = ({ children }: AuthProviderProps) => {
|
||||
|
||||
// Function to check token expiration
|
||||
const checkTokenExpiration = (token: string) => {
|
||||
const decodedToken: any = jwtDecode(token);
|
||||
const currentTime = Date.now();
|
||||
const tokenExpiryTime = decodedToken.exp * 1000; // Convert to milliseconds
|
||||
const timeUntilExpiry = tokenExpiryTime - currentTime;
|
||||
if (!token || typeof token !== 'string') {
|
||||
console.warn("Invalid token provided for decoding");
|
||||
logoutUser();
|
||||
return;
|
||||
}
|
||||
try {
|
||||
const decodedToken: any = jwtDecode(token);
|
||||
const currentTime = Date.now();
|
||||
const tokenExpiryTime = decodedToken.exp * 1000; // Convert to milliseconds
|
||||
|
||||
if (timeUntilExpiry > 0) {
|
||||
setTimeout(logoutUser, timeUntilExpiry); // Auto-logout when token expires
|
||||
} else {
|
||||
logoutUser(); // Immediately logout if token is expired
|
||||
if (tokenExpiryTime > currentTime) {
|
||||
setTimeout(logoutUser, tokenExpiryTime - currentTime); // Auto-logout when token expires
|
||||
} else {
|
||||
logoutUser(); // Immediately logout if token is expired
|
||||
}
|
||||
} catch (error) {
|
||||
console.error("Error decoding token:", error);
|
||||
logoutUser(); // Logout on error
|
||||
}
|
||||
};
|
||||
|
||||
|
||||
useEffect(() => {
|
||||
const storedUser = window.localStorage.getItem('user');
|
||||
if (storedUser) {
|
||||
@@ -104,7 +114,7 @@ const AuthProvider = ({ children }: AuthProviderProps) => {
|
||||
return Promise.reject(error);
|
||||
}
|
||||
);
|
||||
|
||||
|
||||
return (
|
||||
<AuthContext.Provider value={{ state, dispatch }}>
|
||||
{children}
|
||||
|
||||
Reference in New Issue
Block a user