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

Various cleanups and speedups for the Vhosts plugin #13

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

Various cleanups and speedups for the Vhosts plugin #13

wants to merge 10 commits into from

Commits on Feb 4, 2012

  1. Do vhost twiddling *after* all host header cleanups

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    08f738d View commit details
    Browse the repository at this point in the history

  2. Strip port info *before* stripping a trailing period from the domain

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    f83dd6b View commit details
    Browse the repository at this point in the history

  3. Various code cleanups 

    - $uri only used once so replaced usage with $req->request_uri
- $map_name was incorrect description for var contents
- moved $vhost definition to just before it starts being used
- removed needless $vhost empty checks
- replaced  A; while (X) { A; }  with  do { A; } while (X);
- moved comments next to actual code
- add spacing
    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    9144507 View commit details
    Browse the repository at this point in the history

  4. Skip vhost twiddling checks if it hasn't been configured

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    3586c6d View commit details
    Browse the repository at this point in the history

  5. Return error when vhost maps to nonexistent service

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    2150819 View commit details
    Browse the repository at this point in the history

  6. Optimize vhost_selector by refactoring the use of a closure

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    193652a View commit details
    Browse the repository at this point in the history

  7. Allow vhost mappings to be deleted via the management port

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 4, 2012
    Configuration menu
    Copy the full SHA
    216078a View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2012

  1. Merge remote-tracking branch 'upstream/master' into vhost-cleanups

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 5, 2012
    Configuration menu
    Copy the full SHA
    7ce5e51 View commit details
    Browse the repository at this point in the history

Commits on Feb 6, 2012

  1. Remove speedup claim as "lab" testing showed it to be more like 3%

    @jstubbs-bunnings
    jstubbs-bunnings committed Feb 6, 2012
    Configuration menu
    Copy the full SHA
    643e526 View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2012

  1. Merge branch 'master' into vhost-cleanups 

    Conflicts:
	CHANGES
    @jstubbs-bunnings
    jstubbs-bunnings committed Mar 24, 2012
    Configuration menu
    Copy the full SHA
    8b49a86 View commit details
    Browse the repository at this point in the history