StrafesNET/maptest-bot
3
Fork 0
Code Issues Pull Requests Releases Activity

Use alternate API for current user and skip cookie validation

Browse Source
This commit is contained in:
Carter Penterman
2024-08-27 21:28:33 -05:00
parent 36c4a3d791
commit d1066e2737
3 changed files with 25 additions and 7 deletions

6
commands/submit.js

@ -4,7 +4,7 @@ const fs = require('node:fs');
const noblox = require("noblox.js");
const axios = require("axios").default;
const { submissions, commands, cookies } = require("../config/config.js");
const { getAssetInfo, SubmissionColumnsString, createSubmissionLine, getSubmissionLine, validateMapAsset, getValidationMessage } = require("../common.js");
const { getAssetInfo, getCurrentUser, SubmissionColumnsString, createSubmissionLine, getSubmissionLine, validateMapAsset, getValidationMessage } = require("../common.js");
async function robloxUserFromDiscord(id) {
if (isNaN(id)) return undefined;
@ -48,11 +48,11 @@ async function execute(interaction) {
}
const id = interaction.options.getInteger("asset_id", true);
await noblox.setCookie(cookies[game]);
noblox.setCookie(cookies[game], false);
try {
// Check that the bot owns this model
if (!(await noblox.getOwnership(await noblox.getCurrentUser("UserID"), id, "Asset"))) {
if (!(await noblox.getOwnership(await getCurrentUser(), id, "Asset"))) {
const msg = `🚫 The ${game} maptest bot's inventory does not contain this asset (id: \`${id}\`). You must use the /take command first.`;
await interaction.editReply(msg);
return;

6
commands/take.js

@ -1,7 +1,7 @@
const { SlashCommandBuilder } = require('@discordjs/builders');
const noblox = require("noblox.js");
const { cookies, commands, gamePlaces } = require("../config/config.js");
const { getAssetInfo, buyModel, validateMapAsset, getValidationMessage } = require("../common.js");
const { getAssetInfo, buyModel, getCurrentUser, validateMapAsset, getValidationMessage } = require("../common.js");
/**
* @param {import('discord.js').ChatInputCommandInteraction} interaction
@ -15,12 +15,12 @@ async function execute(interaction) {
}
const id = interaction.options.getInteger("asset_id", true);
await noblox.setCookie(cookie);
noblox.setCookie(cookie, false);
let alreadyOwned;
try {
// Check if the bot already owns this asset
alreadyOwned = await noblox.getOwnership(await noblox.getCurrentUser("UserID"), id, "Asset");
alreadyOwned = await noblox.getOwnership(await getCurrentUser(), id, "Asset");
} catch (error) {
if (error.message !== "400 The specified Asset does not exist!") {
throw error;