Skip to content

(fix) Hide phpinfo utility is not available#16229

Closed
nitriques wants to merge 2 commits intocraftcms:5.xfrom
DeuxHuitHuit:nitriques/phpinfo-available
Closed

(fix) Hide phpinfo utility is not available#16229
nitriques wants to merge 2 commits intocraftcms:5.xfrom
DeuxHuitHuit:nitriques/phpinfo-available

Conversation

@nitriques
Copy link
Copy Markdown
Contributor

This simple PR prevents the cp from displaying a page that we know would error out.

On production systems, phpinfo should be disabled anyways.

Let me know if you need me to change anything.

Thanks!

This simple PR prevents the cp from displaying a page that we know would
error out.

On production systems, phpinfo should be disabled anyways.
@brandonkelly
Copy link
Copy Markdown
Member

Thanks for the PR! I fixed this using the existing isSelectable() method for the next v4 and v5 releases (52eb5f6).

@nitriques
Copy link
Copy Markdown
Contributor Author

@brandonkelly Hey thanks! I did search for this kind of api but could not find it !

@brandonkelly
Copy link
Copy Markdown
Member

Craft 4.13.4 and 5.5.4 are out with that change. Thanks again!

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