diff options
| author | Kim Alvefur <zash@zash.se> | 2023-06-10 12:33:58 +0200 |
|---|---|---|
| committer | Kim Alvefur <zash@zash.se> | 2023-06-10 12:33:58 +0200 |
| commit | 960f406662c436762033ce13422eb030d3a52f61 (patch) | |
| tree | 8afeaec73f4f6856ca01b6407e72362fe7bf7735 | |
| parent | cbed7dfdf503b8d56de4ae5ed86e32bfc4e8e60e (diff) | |
| download | prosody-960f406662c436762033ce13422eb030d3a52f61.tar.gz prosody-960f406662c436762033ce13422eb030d3a52f61.zip | |
mod_http: Fix error if 'access_control_allow_origins' is set
Because it changes the type of the 'opt_origins' variable from util.set
to the internal _items table so next time an http app is added an error
"attempt to call a nil value (method 'empty')" is triggered. The value
is not used anywhere else.
Noticed when reviewing uses of the '_items' set property.
Not reported by any users, implying this setting is rarely used.
| -rw-r--r-- | plugins/mod_http.lua | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/plugins/mod_http.lua b/plugins/mod_http.lua index 01bb1f6f..0cee26c4 100644 --- a/plugins/mod_http.lua +++ b/plugins/mod_http.lua @@ -149,7 +149,7 @@ function module.add_host(module) local app_credentials = opt_credentials; local app_origins; if opt_origins and not (opt_origins:empty() or opt_origins:contains("*")) then - opt_origins = opt_origins._items; + app_origins = opt_origins._items; end local function cors_handler(event_data) |