Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues... #35

Open
ghost opened this issue Jan 22, 2018 · 9 comments
Open

Issues... #35

ghost opened this issue Jan 22, 2018 · 9 comments
Assignees

Comments

@ghost
Copy link

ghost commented Jan 22, 2018

This just doesn't seem to work with any modern stack and latest NodeBB/Node. Am I missing something?

Just seems none of the widgets appear at all. Seems to link fine. But that's it.

@yariplus
Copy link
Owner

Is there anything in your nodebb logs?

If the widgets are empty, it may not have successfully linked to the server yet.

@yariplus
Copy link
Owner

I spun up a new instance with the latest NodeBB/Node and widgets worked.

Could be an issue on the Java side.

What Minecraft server type and version are you using and what version of the plugin?

@ghost
Copy link
Author

ghost commented Jan 25, 2018

Latest release alpha... Have tested on both Thermos and latest Spigot. Neither work.

Will try again, since you say you just tested it. Will forward log.

@ghost
Copy link
Author

ghost commented Jan 25, 2018

image

image

Actually installs this time without template errors, however nothing shows up. I add any of the widgets, I don't even get the container. No errors in the NodeBB dev log... other than it can't get my server's status...

2018-01-25T18:35:41.490Z [5136] - info: Error getting status for server NodeDirt

Which is odd, since its working fine. Query works, votifier works.. all works. I figured it may be DNS SRV records even being ignored by whatever query process you are using (as its not native MC), however even after changing the query address to the specific IP:port, it still fails.

No errors.. nothing I can do. Suggestions? Im curious... What version of Node, what version of NodeBB did you test this on?

@ghost
Copy link
Author

ghost commented Jan 25, 2018

The previous issue I had with it just last week, was it crashing user profiles... I can still see some remnants of that fiasco here in my current log:

2018-01-25T18:34:30.827Z [5136] - warn: [plugins/nodebb-plugin-minecraft-integration] Hook filter:user.profileLinks is deprecated, please use filter:user.profileMenu instead.
2018-01-25T18:34:30.827Z [5136] - verbose: [plugins] Loaded plugin: nodebb-plugin-minecraft-integration

Would literally crash my site when someone viewed a profile.

Double checked API key, reset, reassigned in game via /nodebb key, still says no link.

@ghost
Copy link
Author

ghost commented Jan 25, 2018

Update.. found this on MC server log.... only one it shows.

[13:49:12 ERROR]: [NodeBB-Integration] Got Cookie: express.sid=REDACTED; Path=/; Expires=Thu, 08 Feb 2018 18:49:14 GMT; HttpOnly; Secure

Went to compile a fresh version, thinking perhaps the releases just aren't up to date, but all of the main dependent repo's are out of date... Vault, OnTime, Vanish et all.

@yariplus
Copy link
Owner

yariplus commented Jan 27, 2018

Well, it's getting a cookie. That's something.

I'm going to try on a fresh VPS, maybe I missed something.

@yariplus
Copy link
Owner

Okay, I'm sure this is where the problem lies. The socket values are not properly added to the config file.

Manually edit the config file so it looks like below. Then do /nodebb reload, you should get a message saying CONNECTED TO FORUM

General:
  Forum URL: https://rfdtest.yaricraft.com
  Forum Name: 'My Forum'
  Forum API Key: keyfromforumacp==
  SocketIO Address: https://rfdtest.yaricraft.com
  SocketIO Transports:
  - websocket
  - polling
  SocketIO Namespace: plugins.MinecraftIntegration

That should hopefully work for you now.

I'll fix the issues I found.

  • Will not connect if the fully qualified domain isn't used.
  • Doesn't automatically set the socket address to the same address as the forum.
  • Doesn't accept keys with underscores.
  • Doesn't add default socket transports.

@ghost
Copy link
Author

ghost commented Feb 14, 2018

Thank you... I will try this here in the next day, and report back the results...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant