luci-app-nlbwmon: add proper file names for json and csv exports.

Currently, the json and csv exports are just named `data` which is not
intuitive. This adds proper file name extensions using the
Content-disposition HTTP header.

Suggested-by: Ameer Dawood <ameer1234567890@gmail.com>
[slightly reword commit message, squash commits]
Signed-off-by: Jo-Philipp Wich <jo@mein.io>

diff --git a/applications/luci-app-nlbwmon/luasrc/controller/nlbw.lua b/applications/luci-app-nlbwmon/luasrc/controller/nlbw.lua
index a8c577929ab93f6cf732ec06edd3eb88d4f90eee..72e87e8f90579f2e028caf15edbde56fe73be495 100644 (file)
--- a/applications/luci-app-nlbwmon/luasrc/controller/nlbw.lua
+++ b/applications/luci-app-nlbwmon/luasrc/controller/nlbw.lua
@@ -57,6 +57,8 @@ function action_data()
                json = "application/json"
        }
 
+       local filename = "data." .. mtype
+
        local args = { }
        local mtype = http.formvalue("type") or "json"
        local delim = http.formvalue("delim") or ";"
@@ -92,6 +94,7 @@ function action_data()
        end
 
        http.prepare_content(types[mtype])
+       http.header("Content-Disposition", "attachment; filename=\"%s\"" % filename)
        exec("/usr/sbin/nlbw", args, http.write)
 end