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.

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

Current conf does not support path_info #69

Open
andybellenie opened this issue Jan 29, 2021 · 1 comment
Open

Current conf does not support path_info #69

andybellenie opened this issue Jan 29, 2021 · 1 comment

Comments

@andybellenie
Copy link

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
Copy link
Contributor

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

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

No branches or pull requests

2 participants