Skip to content

FIX - should now work with php8.1 (not fully tested...)#29

Closed
dacendo wants to merge 1 commit into
PHProxy:masterfrom
dacendo:master
Closed

FIX - should now work with php8.1 (not fully tested...)#29
dacendo wants to merge 1 commit into
PHProxy:masterfrom
dacendo:master

Conversation

@dacendo
Copy link
Copy Markdown
Contributor

@dacendo dacendo commented Oct 24, 2022

No description provided.

@fortkitty
Copy link
Copy Markdown

Fixes the url entry page completely but something still goes wrong when attempting to proxify an address

@us3r1d us3r1d mentioned this pull request Aug 21, 2025
Comment thread index.php
@@ -1,5 +1,7 @@
<?php


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please delete these extra empty lines.

@miglen
Copy link
Copy Markdown
Member

miglen commented May 27, 2026

Thanks for the patch — your null-trim casts for HTTP_REFERER, response header parsing, and cookie-domain handling were incorporated into #48 with credit as a co-author. The wider PHP 8.5 modernization is now on master. Closing in favor of #48.

@miglen miglen closed this May 27, 2026
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