Skip to content

Commit 7168941

Browse files
committed
is_https should not rely on REQUEST_SCHEME which may still be "http" for SSL connection.
1 parent 161607c commit 7168941

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

library/server/wsf/src/wsf_request.e

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -125,9 +125,7 @@ feature {NONE} -- Initialization
125125
end
126126

127127
--| HTTPS support
128-
if attached meta_string_variable ("REQUEST_SCHEME") as l_scheme and then not l_scheme.is_empty then
129-
is_https := l_scheme.is_case_insensitive_equal_general ("https")
130-
elseif attached execution_environment.item ("HTTPS") as l_https and then not l_https.is_empty then
128+
if attached meta_string_variable ("HTTPS") as l_https and then not l_https.is_empty then
131129
is_https := l_https.is_case_insensitive_equal_general ("on")
132130
or else l_https.is_case_insensitive_equal_general ("yes")
133131
or else l_https.is_case_insensitive_equal_general ("true")
@@ -179,8 +177,8 @@ feature -- Destroy
179177
feature -- Status report
180178

181179
is_https: BOOLEAN
182-
-- Is https scheme or protocol?
183-
--| based on REQUEST_SCHEME, or environment variable HTTPS=on
180+
-- Is https connection?
181+
--| based on meta variable HTTPS=on .
184182

185183
debug_output: STRING_8
186184
do

0 commit comments

Comments
 (0)