Difuse/luci
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

* luci/libs: make treatment of "+" conditional in http.protocol ("+" should not be decoded by urldecode)

Browse source 
* luci/https:	fix breakage introduced by "+" decoding
This commit is contained in:
Jo-Philipp Wich 2008-07-05 20:12:24 +00:00
parent bbe086c9ce
commit 8fcbdb39fe
2 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
libs/http/luasrc/http/protocol.lua
View file

@ -23,14 +23,17 @@ HTTP_URLENC_MAXKEYLEN = 1024 -- maximum allowd size of urlencoded parameter nam
-- Decode an urlencoded string.
-- Returns the decoded value.
function urldecode( str )
function urldecode( str, no_plus )
local function __chrdec( hex )
return string.char( tonumber( hex, 16 ) )
end
if type(str) == "string" then
if not no_plus then
str = str:gsub( "+", " " )
end
str = str:gsub( "%%([a-fA-F0-9][a-fA-F0-9])", __chrdec )
end

4
libs/httpd/luasrc/httpd/handler/file.lua
View file

@ -44,7 +44,7 @@ function Simple.getfile(self, uri)
end
function Simple.handle_get(self, request, sourcein, sinkerr)
local file, stat = self:getfile( self.proto.urldecode( request.env.PATH_INFO ) )
local file, stat = self:getfile( self.proto.urldecode( request.env.PATH_INFO, true ) )
if stat then
if stat.type == "regular" then
@ -95,7 +95,7 @@ function Simple.handle_get(self, request, sourcein, sinkerr)
elseif stat.type == "directory" then
local ruri = request.request_uri:gsub("/$","")
local duri = self.proto.urldecode( ruri )
local duri = self.proto.urldecode( ruri, true )
local root = self.docroot:gsub("/$","")
-- check for index files