lucee / lucee-dockerfiles

Official Lucee Dockerfiles for Docker Hub build images
https://hub.docker.com/u/lucee/
MIT License
85 stars 50 forks source link

Current conf does not support path_info #69

Open andybellenie opened 3 years ago

andybellenie commented 3 years ago

This should be added to the lucee proxy_pass to enable path_info to be passed to tomcat for SES URLs.

# Enable path_info - http://www.lucee.nl/post.cfm/enable-path-info-on-nginx-with-lucee-and-railo
  set $pathinfo "";
  # if the extension .cfm or .cfc is found, followed by a slash and optional extra
  if ($uri ~ "^(.+?\.cf[mc])(/.*)") {
      # remember the filepath without path_info
      set $script $1;
      set $pathinfo $2;
      # rewrite the url to match the filepath wthout path_info
      rewrite ^.+$ $script break;
  }
  # set the custom path_info header
  proxy_set_header XAJP-PATH-INFO $pathinfo;
tomchiverton commented 6 months ago

I don't see a downside to having this in the default image. Makes "pull and run" Just Work for more people ?