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

Slight re-wording/text inserts #651

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

Conversation

YeahJenni
Copy link

Mentioned that COMPOSER_ALLOW_SUPERUSER=1 should be ran when upgrading the panel, and also mentioned how 127.0.0.1 shouldn't be used, unless the user knows what they are doing/has a proxy in place.

Slightly clarified, mentioned that 127.0.0.1 should not be used unless a proxy is in place/user knows what they are doing
@@ -100,7 +100,7 @@ After you've downloaded all of the new files you will need to upgrade the core c
simply run the commands below and follow any prompts.

```bash
composer install --no-dev --optimize-autoloader
Copy link

@WeeskyBDW WeeskyBDW Feb 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I dont think it's a good idea to recommend that because it's a security risk and afaik not needed in 99% of the cases (and if needed then they prob is a file perm issue)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put it there to try and follow through with the first install segment, as that has the same thing too, and I feel it sometimes causes issues for people

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it as change though but if it's added prob add a warning on the impact of that env var

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As composer's own documentation states, composer shouldn't be run as root.
Instead of arbitrarily ignoring the warning, it's perhaps best to guide people to composer's documentation instead, although, I don't believe it should be up to pterodactyl's documentation to solve every single problem unrelated to pterodactyl itself.

https://getcomposer.org/doc/faqs/how-to-install-untrusted-packages-safely.md#why-am-i-seeing-a-do-not-run-composer-as-root-super-user-warning-error-

The catch-all phrase under getting started should apply to this:
If you expect to be able to install this with no understanding of basic linux system administration you should stop and turn around now.

@@ -174,4 +174,4 @@ Allocation is a combination of IP and Port that you can assign to a server. Each

![example image of node allocations](../../.vuepress/public/node_allocations.png)

Type `hostname -I | awk '{print $1}'` to find the IP to be used for the allocation. Alternatively, you can type `ip addr | grep "inet "` to see all your available interfaces and IP addresses. Do not use 127.0.0.1 for allocations.
Type `hostname -I | awk '{print $1}'` to find the IP to be used for the allocation. Alternatively, you can type `ip addr | grep "inet "` to see all your available interfaces and IP addresses. Do not use 127.0.0.1 for allocations, *unless you know what you are doing/using a [Minecraft Proxy](https://pterodactyl.io/community/games/minecraft.html)*.
Copy link

@WeeskyBDW WeeskyBDW Feb 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a more general link to proxy, because even if ptero seems a lot used for Miencraft it's not the only game hosted

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

afaik the most common use for proxies with Pterodactyl is Minecraft, and so far it is the only one with a community guide

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.

3 participants