On 2018/5/3 02:52, Thierry Fournier wrote:
>> On 2 May 2018, at 16:49, Willy Tarreau <w...@1wt.eu> wrote:
>>
>> Hi Thierry,
>>
>> when you have a moment, could you please give a quick look at these
>> patches from Patrick so that I know if I can merge them or not ? There
>> are 2 other ones on the list.
>
> Hi Willy and Patrick,
>
> I check it. I don’t understand why you convert the puid in string.
> You could add directly the ouid integer as is in a Lua variable with
> the function lua_pushinteger().
>
> Thierry
I did this for consistency, as this is how Proxy.uuid behaves. Even
though it could return an integer, it converts it to a string and
returns that instead.

>> Thanks,
>> Willy
>>
>> On Sun, Apr 29, 2018 at 02:23:48PM -0400, Patrick Hemmer wrote:
>>> ---
>>> doc/lua-api/index.rst |  8 ++++++++
>>> src/hlua_fcn.c        | 14 ++++++++++++++
>>> 2 files changed, 22 insertions(+)
>>>
>>>
>>> diff --git a/doc/lua-api/index.rst b/doc/lua-api/index.rst
>>> index 7a77e46ee..cdb285957 100644
>>> --- a/doc/lua-api/index.rst
>>> +++ b/doc/lua-api/index.rst
>>> @@ -925,6 +925,14 @@ Server class
>>>
>>>   This class provides a way for manipulating servers and retrieving 
>>> information.
>>>
>>> +.. js:attribute:: Server.name
>>> +
>>> +  Contain the name of the server.
>>> +
>>> +.. js:attribute:: Server.puid
>>> +
>>> +  Contain the proxy unique identifier of the server.
>>> +
>>> .. js:function:: Server.is_draining(sv)
>>>
>>>   Return true if the server is currently draining sticky connections.
>>> diff --git a/src/hlua_fcn.c b/src/hlua_fcn.c
>>> index a8d53d45b..280d8e5af 100644
>>> --- a/src/hlua_fcn.c
>>> +++ b/src/hlua_fcn.c
>>> @@ -490,6 +490,8 @@ int hlua_listener_get_stats(lua_State *L)
>>>
>>> int hlua_fcn_new_server(lua_State *L, struct server *srv)
>>> {
>>> +   char buffer[10];
>>> +
>>>     lua_newtable(L);
>>>
>>>     /* Pop a class sesison metatable and affect it to the userdata. */
>>> @@ -498,6 +500,18 @@ int hlua_fcn_new_server(lua_State *L, struct server 
>>> *srv)
>>>
>>>     lua_pushlightuserdata(L, srv);
>>>     lua_rawseti(L, -2, 0);
>>> +
>>> +   /* Add server name. */
>>> +   lua_pushstring(L, "name");
>>> +   lua_pushstring(L, srv->id);
>>> +   lua_settable(L, -3);
>>> +
>>> +   /* Add server puid. */
>>> +   lua_pushstring(L, "puid");
>>> +   snprintf(buffer, sizeof(buffer), "%d", srv->puid);
>>> +   lua_pushstring(L, buffer);
>>> +   lua_settable(L, -3);
>>> +
>>>     return 1;
>>> }
>>>
>>>
>

Reply via email to