Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • I ISPConfig 3
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 581
    • Issues 581
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 47
    • Merge requests 47
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ISPConfig
  • ISPConfig 3
  • Merge requests
  • !578

improved nginx http to https redirect

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Ramil Valitov requested to merge ram/ispconfig3:master into master Apr 17, 2017
  • Overview 3
  • Commits 1
  • Pipelines 0
  • Changes 1

"if" constructions in nginx should be used only if no other solution is possible, because "if" is the slowest directive and adds performance penalties ("If is Evil"). This merge request is supposed to improve this issue and adds another "server" block instead of using "if" directive. This is a "standard" approach which is widely used in nginx configurations for http->https redirections. All other functionality and features of ISPConfig are preserved.

Closes #2209 (closed) (redirection without "if" directive) Closes #3118 (closed) (the "split" is achieved using "server" blocks without "if" directive)

Edited Sep 14, 2020 by Marius Burkard
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: master