Whats with this error? [NOBLOX.JS-SERVER]

NachtHemd · November 15, 2020, 2:33pm

Instead of debugging the entire system simultaneously, why dont you have your server return a simple text/plain message when you do a GET request to your domain root to see if your DNS A record is actually set properly.

dollychun · November 15, 2020, 2:35pm

Try ping the domain. It should always time out.

nonraymonable · November 15, 2020, 2:49pm

OK, scrap everything I just said earlier, theres something up with the handleJoinRequest code on my server.

I rewritten the entire code into a function, and got the same error but kinda screwed up (which is more than likely my fault)

Edit: I’m going to reinstall the server, except install a different version and see if that helps.

nonraymonable · November 15, 2020, 3:21pm

Alright, new server installed, and now I’m stuck.

I have a brand new error (somehow) that seems to be a server error, although I can’t put my finger on it.

(Note, this is from my ssh command line, not from ROBLOX STUDIO.)

Heres the current code that runs it

And the new configuration:

Here is what my configuration currently looks like

Home Folder:

Utility Folder:

NuclearTheNoob · November 15, 2020, 3:44pm

If you take a look at the wrap function it is checking for the required arguments. What happens if you directly call noblox handleJoinRequest with correct arguments?

github.com

noblox/noblox.js/blob/master/lib/internal/wrap.js

/*
const args = ['jar', 'username', 'password']

raw function login (jar, username, password)

exported function login ()
  check number of arguments if necessary
    options object: login(arguments[0])
    individual arguments:
      for each argument add to an options array corresponding with argument order
      login(options)
*/
// Define
exports.wrapExport = function (wrapFunction, required, optional) {
  const reqLength = required.length
  if ((reqLength + optional.length) > 0) {
    return function () {
      let options = {}
      const length = arguments.length
      let assume = false

This file has been truncated. show original

nonraymonable · November 15, 2020, 3:45pm

How would I do that? Do I use a POST request tester?

I believe that browsers can only do GET requests

Note that the only real issue is handleJoinRequest, things like promote work fine.

NuclearTheNoob · November 15, 2020, 3:46pm

Directly call it using node. Forget the HTTP stuff for now.

NuclearTheNoob · November 15, 2020, 3:49pm

I think I may have found the issue. handleJoinRequest in noblox.js takes in a userId, whereas the server takes a username. Try changing the requiredFields on the server from ‘username’: ‘string’ to ‘userId’: ‘int’ and making the necessary changes on the lua side.

EDIT: The issue seemingly was an update that changed the args and inadvertently broke the server they wrote.

nonraymonable · November 15, 2020, 3:52pm

Nope, didn’t solve it, it just ruined the server even more lol

NuclearTheNoob · November 15, 2020, 3:53pm

What exactly did you change? You may need to change more.

nonraymonable · November 15, 2020, 3:53pm

I changed everything that referenced username, except in the noblox.js module itself, as I don’t have access to that.

I also put in the UserId.

NuclearTheNoob · November 15, 2020, 3:57pm

Can you paste your new handleJoinRequest server function?

nonraymonable · November 15, 2020, 4:02pm

NuclearTheNoob · November 15, 2020, 4:08pm

Ah you are using the Hamzah one. Can I also see your HandleJoinRequest in Server.mod.lua? As well remove the ${Username} in the strings as the errors are (maybe?) coming from there.

github.com

Hamzah-z/noblox.js-server/blob/master/lua/Server.mod.lua

-- THIS IS A MODULESCRIPT (insert as child of Main)

local Server = {}

--// SERVICES

local HttpService = game:GetService("HttpService")

--// MODULES

local Configs = require(script.Parent.Configs)

--// MAIN
 
local function Request(Function, RequestBody)
	
	--Before sending the request, add our auth_key to the body
	RequestBody["auth_key"] = Configs.AUTH_KEY
	
	local response = HttpService:RequestAsync(

This file has been truncated. show original

nonraymonable · November 15, 2020, 4:09pm

Also, same error when changing the console.log text

NuclearTheNoob · November 15, 2020, 4:12pm

You’ll need to change the username to a userId as well as edit Server.mod.lua (the api bit) and remove the assert typeof and change username to userId.

nonraymonable · November 15, 2020, 4:13pm

Ah, wait I forgot to send the updated image lol

And I removed that, it’s a server problem now lol

NuclearTheNoob · November 15, 2020, 4:18pm

Somewhere it is checking that username is a string and finding out that it doesn’t exist, are you absolutely sure that you changed every reference?

EDIT: Have you edited JoinRequests under validator.js?

nonraymonable · November 15, 2020, 4:25pm

OH MY GOD, THANK YOU!!!

I forgot to change the highlighted so It still errored until I noticed & fixed

Thank you so much!

Conmmander · November 16, 2020, 4:14am

I do not recommend using the Hamza one. It is not regularly maintained. Use the one by Suufi.

Part of your issues is that Hamza’s likely using an outdated version of Noblox, therefore leaving you screwed. Roblox has changed a lot of internals since his was lasted updated.

Use this: