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

Querying endpoint including eggs that use Process Management config inheritance throws exception #4964

Open
3 tasks done
MrJoshuaT opened this issue Dec 31, 2023 · 4 comments
Labels
bug Something that's not working as it's intended to be.

Comments

@MrJoshuaT
Copy link

Current Behavior

Querying an endpoint, such as api/application/servers?include=egg or /api/application/nests/1/eggs throws an internal server error via the API.

[2023-12-31 18:11:24] production.ERROR: Syntax error {"userId":1,"exception":"[object] (JsonException(code: 4): Syntax error at /var/www/pterodactyl/app/Transformers/Api/Application/EggTransformer.php:44) [stacktrace]

Expected Behavior

Endpoints including egg data should not return an error.

Steps to Reproduce

The default Minecraft nest with id 1, seems to only one causing this. Exporting and importing the eggs to a new nest will not throw this error anymore. If I can do anymore debugging steps myside I can do this.

Panel Version

1.11.5

Wings Version

1.11.8

Games and/or Eggs Affected

No response

Docker Image

No response

Error Logs

https://pteropaste.com/325f67a6sric/

Is there an existing issue for this?

  • I have searched the existing issues before opening this issue.
  • I have provided all relevant details, including the specific game and Docker images I am using if this issue is related to running a server.
  • I have checked in the Discord server and believe this is a bug with the software, and not a configuration issue with my specific system.
@MrJoshuaT MrJoshuaT added the not confirmed Report seems plausible but requires additional testing or 3rd part confirmation. label Dec 31, 2023
@MrJoshuaT
Copy link
Author

After much debugging, it seems I created an egg that was for Paper but with optimisations, with this under the Process Management section I used "Copy Settings From" and had Paper selected. When I just copied these values from the source egg to the new egg it didn't produce the error via the API

@Boy132
Copy link
Contributor

Boy132 commented Jan 4, 2024

Can confirm this. Eggs that have set "Copy Settings From" produce an error with the api.
The EggTransformer seems to ignore inherits.

@DevNatan
Copy link

I'm also having this issue, /nest/1 always fails with internal server error. Any other id like /nest/2 works fine

@danny6167
Copy link
Member

Quick note for anyone trying to reproduce this.

Clear the process management config from the child egg otherwise the error wont trigger.

@danny6167 danny6167 added bug Something that's not working as it's intended to be. and removed not confirmed Report seems plausible but requires additional testing or 3rd part confirmation. labels May 5, 2024
@danny6167 danny6167 changed the title Querying endpoint including eggs sometimes throws json exception Querying endpoint including eggs that use Process Management config inheritance throws exception May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that's not working as it's intended to be.
Projects
Status: 🆕 New
Development

No branches or pull requests

4 participants