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

added custom core keeper docker image which uses nss wrapper #264

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Osanchez
Copy link

@Osanchez Osanchez commented Sep 9, 2024

Description

added custom docker image for core keeper game as a requirement to use the libparty.so plugin https://github.com/PlayFab/PlayFabParty. this plugin is used for cross play on core keeper servers. The following plugin on recent release was causing segfault errors. after investigation I found that the cause was a failure on function calls in c lib getuid(); and getpwuid(uid); which relies on access to the /etc/passwd file. due to the nature of the docker containers when using wings, this directory is not available so the command fails. Another game valheim experienced similar issues. Until the latest version of wings is released, which contains fixes for mounting this directory as pointed here pterodactyl/wings#197, this is the workaround.

All Submissions:

  • Have you ensured there aren't other open Pull Requests for the same update or change?
  • Have you created a new branch for your changes and PR from that branch and not from your master branch?

New Image Submissions:

  1. Have you added your image to the Github workflows?
    yes

  2. Have you updated the README list to contain your new image?
    yes

@Osanchez
Copy link
Author

@parkervcp

@QuintenQVD0
Copy link
Collaborator

@parkervcp

no need to ping him, there other people here that can do the same.

I get that this is an issue but pterodactyl and pelican wings have now a build in fix, just that ptero has to release a newer version before anyone actualy will use it.

for me this is fine but a bit redundand for something that will fix itself in the future.

So I will aprove this but parker himself has the final message if he want to merge this or not.

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

Successfully merging this pull request may close these issues.

2 participants