pcapDecoder allows .hex files as input. Make public some Avp methods for serializatio...
[anna.git] / include / anna / diameter / codec / Message.hpp
index 874752b..9f3c85f 100644 (file)
@@ -1,8 +1,8 @@
-// ANNA - Anna is Not 'N' Anymore
+// ANNA - Anna is Not Nothingness Anymore
 //
 // (c) Copyright 2005-2014 Eduardo Ramos Testillano & Francisco Ruiz Rayo
 //
-// https://bitbucket.org/testillano/anna
+// http://redmine.teslayout.com/projects/anna-suite
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions
@@ -14,7 +14,7 @@
 // copyright notice, this list of conditions and the following disclaimer
 // in the documentation and/or other materials provided with the
 // distribution.
-//     * Neither the name of Google Inc. nor the names of its
+//     *  Neither the name of the copyright holder nor the names of its
 // contributors may be used to endorse or promote products derived from
 // this software without specific prior written permission.
 //
@@ -146,7 +146,31 @@ class Message {
   bool flagsOK(int &rc) const throw(); // flags coherence regarding dictionary. Only must be called when Message is identified at the dictionary.
   int addChild(Avp *avp) throw() { return Avp::addChild(a_avps, a_insertionPositionForChilds, avp); }
   const anna::diameter::stack::Command *getStackCommand(CommandId id) const throw(anna::RuntimeException);
-  Avp * addFailedAVP() throw(); // returns Failed-AVP if exists, creates it when missing
+
+  void setFailedAvp(const parent_t &parent, AvpId wrong, const char *wrongName = NULL) throw(anna::RuntimeException);
+         // During message decoding and validation, the first wrong avp is stored and all the tracking is managed to find out its
+      //  nested path for the case of grouped avps with wrong avps inside. Remember the RFC 6733, section 7.5:
+      //
+      //                                  In the case where the offending AVP is embedded within a Grouped AVP,
+         //                               the Failed-AVP MAY contain the grouped AVP, which in turn contains
+         //                               the single offending AVP.  The same method MAY be employed if the
+         //                               grouped AVP itself is embedded in yet another grouped AVP and so on.
+         //                               In this case, the Failed-AVP MAY contain the grouped AVP hierarchy up
+         //                               to the single offending AVP.  This enables the recipient to detect
+         //                               the location of the offending AVP when embedded in a group.
+      //
+      // The first wrong avp found will set the final result code, as the RFC recommends:
+      //
+         //                               The value of the Result-Code AVP will provide information on the reason
+         //                               for the Failed-AVP AVP.  A Diameter answer message SHOULD contain an
+         //                               instance of the Failed-AVP AVP that corresponds to the error
+         //                               indicated by the Result-Code AVP.  For practical purposes, this
+      //                                  Failed-AVP would typically refer to the first AVP processing error
+         //                               that a Diameter node encounters.
+         //
+      // The message keeps the list (reverse order) of avps hierarchy (in case of grouping) for the final Failed-AVP construction,
+      // which is done at the end of decoding or validation, and only the first wrong avp is stored with its corresponding path.
+
 
 protected:
 
@@ -202,9 +226,12 @@ public:
   * Destructor
   */
   ~Message();
-  // Si hago virtual al destructor, al destruir una hija se llama tambien al destructor del padre.
-  // No hace falta porque he hecho virtual a 'clear' que se llama desde el destructor.
-
+  // Virtual destructors are useful when you can delete an instance of a derived class through a pointer to base class:
+  // This destructor is not virtual, then a pointer to base class (even pointing to a children one) will invoke this destructor, not the derived one.
+  // My current solution: virtualizing method 'clear'
+  //
+  // Recommendation:
+  // To sum up, always make base classes' destructors virtual when they're meant to be manipulated polymorphically.
 
   // setters
 
@@ -337,7 +364,12 @@ public:
    Result-Code and/or Failed-AVP AVPs if proceed, but be aware of DIAMETER_COMMAND_UNSUPPORTED Result-Code, because becomes impossible
    to fix (Session-Id SHOULD appear immediately following the Diameter header, and #fix do this manually even if no information about
    the command structure is known, but perhaps another fixed AVPs could not comply... use #getResultCode to find out this situation before
-   using #setStandardToAnswer).
+   using #setStandardToAnswer). Anyway, application could add another Failed-AVP content no detected internally, for example:
+   DIAMETER_CONTRADICTING_AVPS or DIAMETER_INVALID_AVP_BIT_COMBO). Also, application could add more Failed-AVP avps with other
+   wrong avps, or accumulate wrong avps inside the one and only Failed-AVP managed by the stack. The standard is open to add multiple
+   avps inside Failed-AVP or multiple Failed-AVP avps with single or multiple avps inside. This depends on application criteria regarding
+   other nodes. However, internally the Anna::diameter stack only provides one Failed-AVP with the first wrong avp found, as RFC 6733 says
+   in section 7.5.
 
    If application decoding and/or validation operations are ok, user may search for other problems and put the appropiate Result-Code.
    For example, DIAMETER_TOO_BUSY (3004) depends on congestion issues at business layer and cannot be decided with the only message
@@ -402,35 +434,6 @@ public:
   int getResultCode() const throw();
 
 
-  /**
-     Adds a new AVP within a Failed-AVP over an answer message (for requests, do nothing).
-     If Failed-AVP AVP doesn't exists, is added and then filled (added within) with the value provided (empty AVP id representantion).
-     If Failed-AVP AVP already exists, is filled (added within) with the value provided (empty AVP id representantion).
-
-     This method is internally used during #decode and/or #valid procedures in order to build automatic answers.
-
-     @param id Avp identifier as pair (code,vendor-id).
-
-     @return Pointer to the new AVP added within Failed-AVP, to make easy data-part accessif needed.
-  */
-  Avp * setNewFailedAvp(AvpId id) throw(anna::RuntimeException) { if(isRequest()) return NULL; return (addFailedAVP()->addAvp(id)); }
-
-  /**
-     Adds a new AVP within a Failed-AVP over an answer message (for requests, do nothing).
-     If Failed-AVP AVP doesn't exists, is added and then filled (added within) with the value provided (empty AVP id representantion).
-     If Failed-AVP AVP already exists, is filled (added within) with the value provided (empty AVP id representantion).
-
-     This method is internally used during #decode and/or #valid procedures in order to build automatic answers, but application
-     could call this for set another Failed-AVP content no detected by these methods, for example: DIAMETER_CONTRADICTING_AVPS or
-     DIAMETER_INVALID_AVP_BIT_COMBO).
-
-     @param id Avp identifier as pair (code,vendor-id).
-
-     @return Pointer to the new AVP added within Failed-AVP, to make easy data-part accessif needed.
-  */
-  Avp * setNewFailedAvp(Avp *avp) throw() { if(!avp || isRequest()) return NULL; return (addFailedAVP()->addAvp(avp)); }
-
-
   /**
      Adds an avp child providing its identifier and reserve internal memory it.
 
@@ -747,7 +750,7 @@ public:
   std::string asXMLString() const throw();
 
   /**
-     Comparison operator
+     Comparison operator by mean serialization
 
      @param m1 Instance 1 for Message class
      @param m2 Instance 2 for Message class