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

[Bug]: Added users to Pterodactyl then cannot create an account on controlpanel. #977

Open
SneakyHub opened this issue Jun 22, 2024 · 5 comments
Labels
Bug / Fix Something isn't working and may need a fix Medium Priority Needs attention, not urgent

Comments

@SneakyHub
Copy link

What is your request about?

When you create an account on controlpanel, it will create an account in both databases Pterodactyl and Controlpanel. That's all good but if a previous user who created an account on the controlpanel, then goes ahead creates a server then invites their friend or whoever via the users tab in pterodactyl it will then create a user with their email/username within the pterodactyl database. The issue here is if that friend or whoever was to create their own controlpanel account it will say the user already exists in the database even though it doesn't exist in the controlpanel database, it exists in the pterodactyl database. Therefore making it impossible for a user to then use that email for their own personal controlpanel account.

Branch

main

Solution idea

Have a better way to sync pterodactyl users to the controlpanel. Either have a cron that syncs the data between the two databases properly or have a way for that user to sync their pterodactyl account to a dashboard account or have it so when a new user is added to pterodactyl it then uses those same details in the controlpanel database. It can automatically sync once the new user has been added to pterodactyls database so perhaps making it check for new users, or however you would like to go about it.

Ctrlpanel Logs

no logs. just a simple friendly error code. user already exists.

Additional Info

Latest version of controlpanel and pterodactyl as of 22/08/2024.
(no modified code, no themes, no addons, pterodactyl and controlpanel are stock)
This issue affects all controlpanel versions.

@S0ly S0ly added Bug / Fix Something isn't working and may need a fix High Priority Critical. Act quickly for resolution labels Jun 22, 2024
@SneakyHub
Copy link
Author

This was posted a week ago and has no response. Is there any update with this? Any progress? Or at least can you confirm this is an issue. Again it's been a week since I've posted this and I haven't heard anything. Can you give me an update please. Thanks.

@S0ly
Copy link
Member

S0ly commented Jul 1, 2024

Hi,

I tagged this issue as high priority, but it could potentially be a medium priority – I'm not entirely sure.

As you know, this is an open source project, and we're all volunteers doing our best. Currently, this issue isn't at the top of our priority list, and it hasn't been fully confirmed or reviewed yet.

Our focus right now is on resolving other "blocking" issues to get version 0.10 released. Wen we can, we'll review this issue and decide whether to add it to our to-do list.

In the meantime, you're welcome to submit a pull request to fix the issue if you'd like!

@1day2die
Copy link
Collaborator

1day2die commented Jul 1, 2024

its an issue that could be regarded before the release of 0.10 eventhough i would label it as medium priority as this is kinda an edge case

@S0ly S0ly added Medium Priority Needs attention, not urgent and removed High Priority Critical. Act quickly for resolution labels Jul 1, 2024
@lezetho
Copy link

lezetho commented Nov 2, 2024

This could work, we would just have to adjust the cronjob to also sync to Pterodactyl. However the main issue is how the passwords would sync between platforms (haven't fully taken a look yet so I'm unsure if it's possible or not)

@1day2die
Copy link
Collaborator

This could work, we would just have to adjust the cronjob to also sync to Pterodactyl. However the main issue is how the passwords would sync between platforms (haven't fully taken a look yet so I'm unsure if it's possible or not)

both systems use the same hashing method, theoretically, you can just send the hashed password from ctrlpanel to ptero via the API. this is also done during the installation process

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug / Fix Something isn't working and may need a fix Medium Priority Needs attention, not urgent
Projects
None yet
Development

No branches or pull requests

4 participants