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

srcds Module is broken #1270

Open
3 tasks done
K1ller0561 opened this issue Dec 9, 2024 · 18 comments
Open
3 tasks done

srcds Module is broken #1270

K1ller0561 opened this issue Dec 9, 2024 · 18 comments
Labels

Comments

@K1ller0561
Copy link

Operating System

Debian 12

AMP Version and Build Date

2.6.0.6 - 20241208.1

AMP Release Stream

Development

I confirm that

  • I have searched for an existing bug report for this issue.
  • I am using the latest available version of AMP.
  • my operating system is up-to-date.

Intended Action

I try so start my srcds Module Servers (CSS, TF2, L4D2, GMOD, etc.) but this error comes: Source RCON failed to disconnect: Object reference not set to an instance of an object.

Expected Behaviour

That the srcds module is repaired. So a fix on the part of the developers.

Actual Behaviour

See log:

[05:01:05] [srcds Debug/11] : Working Directory: ./srcds/232330
[05:01:05] [srcds Debug/11] : Arguments: -game cstrike -console -usercon -ip 138.201.33.171 -port 10001 +rcon_password ********** +map surf_home_sweet_home_v4 +maxplayers 10 +hostname Server1 +sv_pure -1 +log on -tickrate 67 +logaddress_add 138.201.33.171:56897
[05:01:05] [srcds Debug/11] : Application state changed from PreStart to Starting
[05:01:05] [System Info/11] : AMP is up to date.
[05:01:07] [srcds Info/15] : Source RCON disconnecting...
[05:01:07] [srcds Error/15] : Source RCON failed to disconnect:
Object reference not set to an instance of an object.
[05:01:07] [RCON Error/15] : NullReferenceException
[05:01:07] [RCON Error/15] : [0] (NullReferenceException) : Object reference not set to an instance of an object.
[05:01:07] [RCON Error/15] : at RCONPlugin.SourceRconClient.Disconnect()
[05:01:07] [srcds Debug/15] : Application state changed from Starting to Stopped
[05:01:07] [System Warning/15] : Server failed to start 1/5

Reproduction

This problem is present since i upgraded to AMP 2.6, in AMP 2.5 is the srcds module running fine.

@K1ller0561 K1ller0561 added the bug label Dec 9, 2024
@IceOfWraith
Copy link
Collaborator

Click the update icon for the instance from the main ADS screen in your browser (where you see all the instances), then update the game server inside the instance. Start it again and share the entire log file, please.

@K1ller0561
Copy link
Author

AMPLOG 2024-12-09 14-27-07.log

Steps are done, same problem …

@IceOfWraith
Copy link
Collaborator

The RCON error is due to the game server stopping so quickly. Something else is causing it to fail in the first place. I'd recommend running in a Docker container as it could be dependency related.

@Greelan
Copy link
Contributor

Greelan commented Dec 9, 2024

That's not it, I've run the srcds module in docker and have had the same issues. Is AMP setting the PATH env var as needed for srcds servers on Linux?

@IceOfWraith
Copy link
Collaborator

When did this start becoming an issue?

@Greelan
Copy link
Contributor

Greelan commented Dec 9, 2024

2.6, as noted by the OP

@IceOfWraith
Copy link
Collaborator

Yeah, I saw what they had, but you seemed to have more insight, so I was curious if it was something changed in AMP or in SRCDS that AMP needed to accommodate.

@Greelan
Copy link
Contributor

Greelan commented Dec 9, 2024

Generic module templates continue to run, so no change in srcds itself

@Greelan
Copy link
Contributor

Greelan commented Dec 9, 2024

Including templates for servers - like CSS and FOF - that fail in AMP

@K1ller0561
Copy link
Author

Something must have been changed in AMP 2.6 that did not cause any errors in version 2.5. The srcds module has always run without errors.

@IceOfWraith
Copy link
Collaborator

The majority of AMP's foundation was changed in 2.6. So it's not surprising it started then. The question is what's not working with it. But I'll figure that out!

@K1ller0561
Copy link
Author

K1ller0561 commented Dec 10, 2024

The error message "Object reference not set to an instance of an object" typically occurs when you try to access a property or method on an object that is currently null.

#1096
same problem, but a csgo server.
i can't found this file in other srcds games.

@IceOfWraith
Copy link
Collaborator

Yeah that's unrelated. I just need to get to this bug. Dealing with another issue first.

@K1ller0561
Copy link
Author

Ok, great. Thanks in advance for troubleshooting. I hope that it will be found. If there is anything new, please let me know 😀

@IceOfWraith
Copy link
Collaborator

I just want to note, Greelan made generic templates that are more up to date compared to the SRCDS module version for many of these game servers. If you create a new instance, it should be his instance type and still works fine. This should be done regardless since SRCDS module is being deprecated over time. But especially now since the module isn't working while I work to figure out the cause of this.

@IceOfWraith IceOfWraith changed the title srcds Module is broken - Object reference not set to an instance of an object. srcds Module is broken Dec 13, 2024
@cwagz
Copy link

cwagz commented Dec 15, 2024

I just want to note, Greelan made generic templates that are more up to date compared to the SRCDS module version for many of these game servers. If you create a new instance, it should be his instance type and still works fine. This should be done regardless since SRCDS module is being deprecated over time. But especially now since the module isn't working while I work to figure out the cause of this.

Will I get the correct template if I create a new instance using Counter-Strike: Source? When I go to create a new instance I see Counter-Strike:Source listed twice. Not sure how to tell which one is using the updated template that works.

@IceOfWraith
Copy link
Collaborator

One of those is the SRCDS module. In the next update that will only list one. If you make one of the instances, you can right click and Browse Datastore. If there's a file named GenericModule.kvp you have the newer one.

@Greelan
Copy link
Contributor

Greelan commented Dec 15, 2024

Should be the second one in the list

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

No branches or pull requests

4 participants