Software / code / prosody
Diff
plugins/mod_http.lua @ 13139:5d5869f14c4d 0.12
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.
| author | Kim Alvefur <zash@zash.se> |
|---|---|
| date | Sat, 10 Jun 2023 12:33:58 +0200 |
| parent | 12923:419e55abd285 |
| child | 13141:451cb119026e |
line wrap: on
line diff
--- a/plugins/mod_http.lua Sat Jun 10 12:14:12 2023 +0200 +++ b/plugins/mod_http.lua Sat Jun 10 12:33:58 2023 +0200 @@ -149,7 +149,7 @@ 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)
