Ack Comment: 1. Note that logMaxLogrecsize is a setting in the log service configuration object that can be changed in runtime. The default setting is 1024. If the log record becomes longer than logMaxLogrecsize it will be truncated. 2. The logMaxLogrecsize is for the whole log record including the part created based on other tokens than @Cb (represents the log message corresponding to additional text). This means that even if logMaxLogrecsize is set to 65535 an additional text of length 65535 in a notification will be truncated.
Thanks Lennart > -----Original Message----- > From: [email protected] [mailto:[email protected]] > Sent: den 8 januari 2016 14:38 > To: Lennart Lund; Minh Chau H > Cc: [email protected] > Subject: [PATCH 1 of 1] ntfs: increase limit of additionalText while logging > notification[#1660] > > osaf/libs/common/ntfsv/include/ntfsv_msg.h | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > > While logging alarm notifications having additionalText > 1024, Ntf server > truncates > additionalText to 1024. > > There is no problem with delievery of notifications having additionalText > >1024, > only saflogging of alarm and security alram notifiation is affected. > > Patch ensures that MAX length of addtionaltext conforms to MAX value of > logMaxLogrecsize as mentioned in LOGSV PR doc Section 3.5.2.1. > > diff --git a/osaf/libs/common/ntfsv/include/ntfsv_msg.h > b/osaf/libs/common/ntfsv/include/ntfsv_msg.h > --- a/osaf/libs/common/ntfsv/include/ntfsv_msg.h > +++ b/osaf/libs/common/ntfsv/include/ntfsv_msg.h > @@ -31,7 +31,9 @@ extern "C" { > #define NTFS_MAX_STRING_LEN 64 > > /* Common notification limits TODO: specifify */ > -#define MAX_ADDITIONAL_TEXT_LENGTH 1024 > +/*MAX length of addtionaltext conforms to MAX value of logMaxLogrecsize > +as mentioned in LOGSV PR doc Section 3.5.2.1*/ > +#define MAX_ADDITIONAL_TEXT_LENGTH 65535 > #define MAX_DISCARDED_NOTIFICATIONS 1024 > > /* Message type enums */ ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
