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

novetus:// URI seems to be ignoring all passed paramters #71

Open
breadtf opened this issue May 13, 2024 · 9 comments
Open

novetus:// URI seems to be ignoring all passed paramters #71

breadtf opened this issue May 13, 2024 · 9 comments

Comments

@breadtf
Copy link

breadtf commented May 13, 2024

When I attempt to use the novetus:// URI to open novetus from a web link, it appears to completly ignore the extra connection info I pass it.

Below is an example of this. The first line is the decoded strings (With an HTTP header I forgot to remove), second is the actual encoded strings which is passed to the client. This screenshot also shows the novetus client completly ignoring them
image

I'm not even sure if this is the "correct" way to do this due to the lack of any documentation for the server/URI.

Any help or links to documentation would be appreciated

(Note: I'm not generating them/copying them from the server, I'm attempting to write this myself from the code avalible here

@breadtf
Copy link
Author

breadtf commented May 13, 2024

Attempting to manually decode a server-generated string just results in garbage data.
image

@breadtf
Copy link
Author

breadtf commented May 13, 2024

--outputinfo in NovetusCMD appears to be completly non-functional (At least without a master server, which I do not have set-up yet)

@breadtf breadtf closed this as completed May 13, 2024
@breadtf
Copy link
Author

breadtf commented May 13, 2024

After attempting to use the "offical" method of using a self-hosted master server, this issue still appears to happen.

Did a recent update break the URI or something?

@breadtf breadtf reopened this May 13, 2024
@breadtf
Copy link
Author

breadtf commented May 14, 2024

Here is a video showing the issue:

out.mp4

This is most likely a problem with the URI as the master server and game server are getting and sending the correct infomation, which can be seen in these logs:

[200]: GET /list.php?name=Novetus&ip=[IP]&port=53641&client=2011M&version=1.3%20v3.2024.2&id=o8TxXYLl2f6097Zw0Q0GC3L3NnZOym357
[200]: GET /list.php?name=Novetus&ip=[IP]&port=53642&client=2011M&version=1.3%20v3.2024.2&id=LcJbwZipskXji7b1bMhq6LD0NMZSNV5862525
[200]: GET /list.php?name=Novetus&ip=[IP]&port=53643&client=2011M&version=1.3%20v3.2024.2&id=R3XLkUtIhzfT5m2fyqXLiSYZuVaIlE5458

And also in the first comment, where I show the HTTP header for opening the URI.

@breadtf
Copy link
Author

breadtf commented May 15, 2024

Upon further testing in the latest unstable release, it appears to only ignore the port in the Local URI, while the Online URI appears to work fine in this version.
I cant test this with my code, however, as NovetusCMD has completly vanished from the bin folder.

@Bitl
Copy link
Member

Bitl commented May 15, 2024

NovetusCMD is integrated into Novetus now and most URI issues have been fixed as i recall as of the latest snapshot.

Make sure you use the link Novetus provides you in the server info.

@Bitl
Copy link
Member

Bitl commented May 15, 2024

Upon further testing in the latest unstable release, it appears to only ignore the port in the Local URI, while the Online URI appears to work fine in this version. I cant test this with my code, however, as NovetusCMD has completly vanished from the bin folder.

it does not ignore the port, both uris feature the port especially since it's required with the rewrite of the ip/port system

@Bitl
Copy link
Member

Bitl commented May 15, 2024

Upon further testing in the latest unstable release, it appears to only ignore the port in the Local URI, while the Online URI appears to work fine in this version. I cant test this with my code, however, as NovetusCMD has completly vanished from the bin folder.

it does not ignore the port, both uris feature the port especially since it's required with the rewrite of the ip/port system

yes, it does use the port
image

@Bitl Bitl closed this as completed May 15, 2024
@Bitl
Copy link
Member

Bitl commented May 15, 2024

didn't mean to close it

Run novetus with -cmdonly to run Novetus CMD in the snapshot.

@Bitl Bitl reopened this May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants