]> source.dussan.org Git - rspamd.git/commitdiff
[Rework] get rid of util:parse_addr duplicating the util:parse_mail_address, replace...
authorJan Smutny <js@excello.cz>
Fri, 17 Apr 2020 11:50:50 +0000 (13:50 +0200)
committerJan Smutny <js@excello.cz>
Fri, 17 Apr 2020 12:32:55 +0000 (14:32 +0200)
src/lua/lua_util.c
src/plugins/lua/spamassassin.lua
utils/sa_trivial_convert.lua

index f33a46d0d0f4c97f044ea061588e9269a84bb5c0..d2efc9cb2dd3540bfb2367e121fe477f02aefcf7 100644 (file)
@@ -152,29 +152,6 @@ LUA_FUNCTION_DEF (util, parse_html);
  */
 LUA_FUNCTION_DEF (util, levenshtein_distance);
 
-/***
- * @function util.parse_addr(str, [pool])
- * Parse rfc822 address to components. Returns a table of components:
-   *
- * - `name`: name of address (e.g. Some User)
- * - `addr`: address part (e.g. user@example.com)
- * - `user` - user part (if present) of the address, e.g. `blah`
- * - `domain` - domain part (if present), e.g. `foo.com`
- * - `flags` - table with following keys set to true if given condition fulfilled:
- *   - [valid] - valid SMTP address in conformity with https://tools.ietf.org/html/rfc5321#section-4.1.
- *   - [ip] - domain is IPv4/IPv6 address
- *   - [braced] - angled `<blah@foo.com>` address
- *   - [quoted] - quoted user part
- *   - [empty] - empty address
- *   - [backslash] - user part contains backslash
- *   - [8bit] - contains 8bit characters
- **
- * @param {string} str input string
- * @param {rspamd_mempool} pool memory pool to use
- * @return {table/tables} parsed list of mail addresses 
- */
-LUA_FUNCTION_DEF (util, parse_addr);
-
 /***
  * @function util.fold_header(name, value, [how, [stop_chars]])
  * Fold rfc822 header according to the folding rules
@@ -669,7 +646,6 @@ static const struct luaL_reg utillib_f[] = {
        LUA_INTERFACE_DEF (util, tanh),
        LUA_INTERFACE_DEF (util, parse_html),
        LUA_INTERFACE_DEF (util, levenshtein_distance),
-       LUA_INTERFACE_DEF (util, parse_addr),
        LUA_INTERFACE_DEF (util, fold_header),
        LUA_INTERFACE_DEF (util, is_uppercase),
        LUA_INTERFACE_DEF (util, humanize_number),
@@ -1487,51 +1463,6 @@ lua_util_levenshtein_distance (lua_State *L)
        return 1;
 }
 
-static gint
-lua_util_parse_addr (lua_State *L)
-{
-       LUA_TRACE_POINT;
-       GPtrArray *addrs;
-       gsize len;
-       const gchar *str = luaL_checklstring (L, 1, &len);
-       rspamd_mempool_t *pool;
-       gboolean own_pool = FALSE;
-
-       if (str) {
-
-               if (lua_type (L, 2) == LUA_TUSERDATA) {
-                       pool = rspamd_lua_check_mempool (L, 2);
-
-                       if (pool == NULL) {
-                               return luaL_error (L, "invalid arguments");
-                       }
-               }
-               else {
-                       pool = rspamd_mempool_new (rspamd_mempool_suggest_size (),
-                                       "lua util", 0);
-                       own_pool = TRUE;
-               }
-
-               addrs = rspamd_email_address_from_mime (pool, str, len, NULL);
-
-               if (addrs == NULL) {
-                       lua_pushnil (L);
-               }
-               else {
-                       lua_push_emails_address_list (L, addrs, 0);
-               }
-
-               if (own_pool) {
-                       rspamd_mempool_delete (pool);
-               }
-       }
-       else {
-               lua_pushnil (L);
-       }
-
-       return 1;
-}
-
 static gint
 lua_util_fold_header (lua_State *L)
 {
index 2ba0b6228a5a9e36d48779d390616f463064c561..798578856f522f247f8421be0af02fd08fa1fc64 100644 (file)
@@ -232,7 +232,7 @@ local function handle_header_def(hline, cur_rule)
       fun.each(function(func)
           if func == 'addr' then
             cur_param['function'] = function(str)
-              local addr_parsed = util.parse_addr(str)
+              local addr_parsed = util.parse_mail_address(str)
               local ret = {}
               if addr_parsed then
                 for _,elt in ipairs(addr_parsed) do
@@ -246,7 +246,7 @@ local function handle_header_def(hline, cur_rule)
             end
           elseif func == 'name' then
             cur_param['function'] = function(str)
-              local addr_parsed = util.parse_addr(str)
+              local addr_parsed = util.parse_mail_address(str)
               local ret = {}
               if addr_parsed then
                 for _,elt in ipairs(addr_parsed) do
index 4725dabd5979f91911410fa49ffdb41b04293829..56a01adfaa4bfd862c3d547823e84a23ca193b70 100644 (file)
@@ -52,7 +52,7 @@ local function handle_header_def(hline, cur_rule)
       fun.each(function(func)
           if func == 'addr' then
             cur_param['function'] = function(str)
-              local addr_parsed = util.parse_addr(str)
+              local addr_parsed = util.parse_mail_address(str)
               local ret = {}
               if addr_parsed then
                 for _,elt in ipairs(addr_parsed) do
@@ -66,7 +66,7 @@ local function handle_header_def(hline, cur_rule)
             end
           elseif func == 'name' then
             cur_param['function'] = function(str)
-              local addr_parsed = util.parse_addr(str)
+              local addr_parsed = util.parse_mail_address(str)
               local ret = {}
               if addr_parsed then
                 for _,elt in ipairs(addr_parsed) do