rack-devel archive mirror (unofficial) https://groups.google.com/group/rack-devel
 help / color / mirror / code / Atom feed
* HTTPS environment variable in uppercase
@ 2011-01-13 11:52 castorosaggio
  2011-01-20 15:03 ` Lee Hambley
  0 siblings, 1 reply; 2+ messages in thread
From: castorosaggio @ 2011-01-13 11:52 UTC (permalink / raw)
  To: Rack Development

Is it possible to introduce the following change?

From:

"rack.url_scheme" => ["yes", "on", "1"].include?(env["HTTPS"]) ?
"https" : "http"


 To:

"rack.url_scheme" => ["yes", "on", "1"].include?
(env["HTTPS"].downcase) ? "https" : "http"


In my platform the values of HTTPS are "ON"/"OFF" rather than
"on"/"off".


 Thanks,

    Andrea

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: HTTPS environment variable in uppercase
  2011-01-13 11:52 HTTPS environment variable in uppercase castorosaggio
@ 2011-01-20 15:03 ` Lee Hambley
  0 siblings, 0 replies; 2+ messages in thread
From: Lee Hambley @ 2011-01-20 15:03 UTC (permalink / raw)
  To: rack-devel

[-- Attachment #1: Type: text/plain, Size: 111 bytes --]

Andrea,

Why not submit a patch with tests, then there's no reason for your request
to not be accepted.

- Lee

[-- Attachment #2: Type: text/html, Size: 171 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-01-20 15:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-13 11:52 HTTPS environment variable in uppercase castorosaggio
2011-01-20 15:03 ` Lee Hambley

Code repositories for project(s) associated with this inbox:

	https://80x24.org/mirrors/rack.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).