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.

Sign up for GitHub

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

Jump to bottom

Change $userrequest source in redirect() #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Change $userrequest source in redirect() #5

wants to merge 1 commit into from

Commits on Apr 1, 2019

  1. Change $userrequest source in redirect() 

    While testing a site that uses a mixed HTTP/HTTPS setup, I ran into an issue here related to the get_protocol() function. It seems to me that neither get_protocol() nor get_address() are necessary, as PHP provides $_SERVER['REQUEST_URI'], which has already the name of the requested page without hostname or protocol. Please consider merging this change to further simplify this awesome plugin and add support for even more sites (e.g. those with multiple server aliases, etc.)
    @MrAlexWeber
    MrAlexWeber committed Apr 1, 2019
    Configuration menu
    Copy the full SHA
    4b59698 View commit details
    Browse the repository at this point in the history