ZABBIX BUGS AND ISSUES
  1. ZABBIX BUGS AND ISSUES
  2. ZBX-7288

proxy tries to process failed response from server

    Details

      Description

      if proxy asks for config data from the server but there is no proxy with such name, previously we would log a message "Proxy host name might not be matching that on the server."

      now it's :

      979:20131105:160656.160 Received configuration data from server. Datalen 23
      979:20131105:160656.160 failed to update local proxy configuration copy: Can't open JSON object or array ""failed"}"

      even the first message here should not appear, it should only be logged when actual received data is ok

        Activity

        Hide
        Andris Mednis added a comment -

        Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-7288 .

        Show
        Andris Mednis added a comment - Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-7288 .
        Hide
        Andris Zeila added a comment - - edited

        (1) It would be nice to send back the error description in info field and write it into logs.
        Andris Mednis It was decided that sending the error description and writing it into receiver's log should be implemented for all types of data transfer between active proxy and server.

        Andris Zeila CLOSED

        Show
        Andris Zeila added a comment - - edited (1) It would be nice to send back the error description in info field and write it into logs. Andris Mednis It was decided that sending the error description and writing it into receiver's log should be implemented for all types of data transfer between active proxy and server. Andris Zeila CLOSED
        Hide
        Andris Mednis added a comment - - edited

        (2) It was decided that the error description should use dynamically allocated memory instead of a fixed size 256-byte buffer.
        Andris Mednis Available in svn://svn.zabbix.com/branches/dev/ZBX-7288 r40883.
        Andris Mednis CLOSED

        Show
        Andris Mednis added a comment - - edited (2) It was decided that the error description should use dynamically allocated memory instead of a fixed size 256-byte buffer. Andris Mednis Available in svn://svn.zabbix.com/branches/dev/ZBX-7288 r40883. Andris Mednis CLOSED
        Hide
        Andris Mednis added a comment -

        Reopening because requirements have changed.

        Show
        Andris Mednis added a comment - Reopening because requirements have changed.
        Hide
        Andris Mednis added a comment -

        Available in the development branch svn://svn.zabbix.com/branches/dev/ZBX-7288.

        Show
        Andris Mednis added a comment - Available in the development branch svn://svn.zabbix.com/branches/dev/ZBX-7288.
        Hide
        Andris Zeila added a comment - - edited

        (3) Regarding zbx_recv_response_dyn(zbx_sock_t *sock, char **info, char **error, int timeout) function. I'm not sure about changing the name - usually _dyn is appended if we have already function working with static buffers with the same name.

        But the error output parameter usually is the last one. I think it would be good to keep this pattern.
        Andris Mednis RESOLVED in r40878.

        Andris Zeila CLOSED

        Show
        Andris Zeila added a comment - - edited (3) Regarding zbx_recv_response_dyn(zbx_sock_t *sock, char **info, char **error, int timeout) function. I'm not sure about changing the name - usually _dyn is appended if we have already function working with static buffers with the same name. But the error output parameter usually is the last one. I think it would be good to keep this pattern. Andris Mednis RESOLVED in r40878. Andris Zeila CLOSED
        Hide
        Andris Zeila added a comment - - edited

        (4) Please review my changes in r40858
        Andris Mednis Thanks! Reviewed and accepted. Minor improvements in r40873.

        Andris Zeila CLOSED

        Show
        Andris Zeila added a comment - - edited (4) Please review my changes in r40858 Andris Mednis Thanks! Reviewed and accepted. Minor improvements in r40873. Andris Zeila CLOSED
        Hide
        Andris Zeila added a comment - - edited

        (5) Warning messages should keep <variable>:<value> pattern instead of <variable>=<value>
        Andris Mednis RESOLVED in r40883.

        Andris Zeila CLOSED

        Show
        Andris Zeila added a comment - - edited (5) Warning messages should keep <variable>:<value> pattern instead of <variable>=<value> Andris Mednis RESOLVED in r40883. Andris Zeila CLOSED
        Hide
        Andris Zeila added a comment -

        Successfully tested

        Show
        Andris Zeila added a comment - Successfully tested
        Hide
        Andris Mednis added a comment - - edited

        (6) Added an error message for active/passive proxy misconfiguration case in r40922.

        Andris Zeila CLOSED

        Show
        Andris Mednis added a comment - - edited (6) Added an error message for active/passive proxy misconfiguration case in r40922. Andris Zeila CLOSED
        Hide
        Andris Mednis added a comment - - edited

        (7) get_proxy_id() changed to get_active_proxy_id() in r40952.

        Andris Zeila CLOSED

        Show
        Andris Mednis added a comment - - edited (7) get_proxy_id() changed to get_active_proxy_id() in r40952. Andris Zeila CLOSED
        Hide
        Andris Zeila added a comment -

        Successfully tested

        Show
        Andris Zeila added a comment - Successfully tested
        Hide
        Andris Mednis added a comment -

        Fixed in versions pre-2.2.2 rev. 40975 and pre-2.3.0 rev. 40977.

        Show
        Andris Mednis added a comment - Fixed in versions pre-2.2.2 rev. 40975 and pre-2.3.0 rev. 40977.
        Hide
        richlv added a comment -
        Show
        richlv added a comment - (8) this was also listed by andris at https://www.zabbix.com/documentation/2.2/manual/introduction/whatsnew222#daemon_improvements -> CLOSED
        Hide
        richlv added a comment -

        reopen to restore label

        Show
        richlv added a comment - reopen to restore label

          People

          • Assignee:
            Unassigned
            Reporter:
            richlv
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: