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

Fix error 500 #21

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

Fix error 500 #21

wants to merge 3 commits into from

Conversation

JrSchmidtt
Copy link
Contributor

No description provided.

@o-be-one
Copy link

o-be-one commented Jul 4, 2022

Hello,
can you tell me why you add a nginx part while this is already supported in the Pterodactyl official repository iirc? I was pretty sure here the goal is to support Traefik.
Thanks

@JrSchmidtt
Copy link
Contributor Author

Nginx is in the panel core in the latest version. The panel does not go up without nginx . Traefik can be used normally on top of it.
But it's just a suggestion, I particularly had to make these adjustments for the panel to work in your docker.

@emilyastranova
Copy link

I ended up removing Traefik entirely and just used my existing Nginx server to handle SSL because of the 500 error and then the ACME errors flooding the Trafeik container logs

@Zerka30
Copy link

Zerka30 commented Jan 28, 2023

Btw I tried to docker-compose run --rm panel php artisan key:generate but keep having issues with :

ln: /app/storage/logs/logs: File exists
external vars exist.
Checking if https is required.
Checking if letsencrypt email is set.
No letsencrypt email is set using http config.
Removing the default nginx config
DB_PORT not specified, defaulting to 3306
Checking database status.
database (172.20.0.3:3306) open
Migrating and Seeding D.B

In EncryptionServiceProvider.php line 101:

  No application encryption key has been specified.


Starting cron jobs.
Starting supervisord.

In EncryptionServiceProvider.php line 101:

  No application encryption key has been specified.


ERROR: 1

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.

4 participants