osgi / bugzilla-archive

Archive of OSGi Alliance Specification Bugzilla bugs. The Specification Bugzilla system was decommissioned with the move to GitHub. The issues in this repository are imported from the Specification Bugzilla system for archival purposes.
0 stars 1 forks source link

[RFC-149] common minor issues #1752

Closed bjhargrave closed 13 years ago

bjhargrave commented 13 years ago

Original bug ID: BZ#1884 From: Evgeni Grigorov <e.grigorov@prosyst.com> Reported version: R4 V4.3

bjhargrave commented 13 years ago

Comment author: Evgeni Grigorov <e.grigorov@prosyst.com>

  1. The requirements numbering needs update current: REQUIREMENT[8] should be: REQUIREMENT[1]
  2. Fig. 1 title is out of date current: Basic Architecture of TR-069 Protocol Implementation . should be: Basic architecture of TR-069 Enabled OSGi implementation
  3. Actually the figure 3 is 2 current: Fig.3 Sequence Examples of RPCs including Response and atomic session management should be: Fig.2 Sequence Examples of RPCs including Response and atomic session management
  4. The table 4 caption is wrong current: Table 4:Mapping from the TR-069 data type to DmtData Format for GetParameterValues RPC for “boolean”, “int”, “unsignedInt”, “long”, “unsignedLong”, and “dateTime”. shold be: Table 4:Mapping from the TR-069 data type to DmtData Format for SetParameterValues RPC for “boolean”, “int”, “unsignedInt”, “long”, “unsignedLong”, and “dateTime”.
  5. the PAsends -> 9. the PA sends
  6. "Mapping of SetParameterValues RPC to Dmt Admin interfaces", the potential verification in 4.f. is duplicated to 4.a. The suggestion is that we can remove the second one.
  7. "Mapping of SetParameterValues RPC to DMT Admin Interfaces" current: if the usedsession is atomic, go to Step 5. should be: if the used session is atomic, go to Step 6.
  8. current: III. If the Acl of all leaf nodes allows it, the parameter is assumed as writable. go to Step 2.e.IV. should be: III. If the Acl of all leaf nodes allows it, the parameter is assumed as writable. Go to Step 2.e.IV..

  9. Mapping of AddObject RPC to Dmt Admin interfaces, step 3 Step 3. If not, it is an error condition. - it's not clear how to handle that error condition
  10. Table 2 title is wrong current: Mapping of TR-069 ACS RPCs to DMT Admin interfaces shoudl be: Mapping of DMT Admin interfaces to TR-069 ACS RPCs
  11. Section 6.3.1, Opening Session current: Note that if the SetParameterValues contains only one ParameterValueStruct with the type of “string” and the type of the node in DMT corresponding to the parameter name is DmtConstant.DDF_SUBTREE_URI, the PA may need to write multiple node values. should be: Note that if the SetParameterValues contains only one ParameterValueStruct with the type of “string” and the type of the node in DMT corresponding to the parameter name is DmtConstant.DDF_LIST_SUBTREE, the PA may need to write multiple node values.
  12. ParameterValueStrusts -> ParameterValueStruscts
  13. current: session as described in Section 6.4.1. In that case, the session should be opened at the start of processing the request that needs to write node values, should be: session as described in Section 6.2. In that case, the session should be opened at the start of processing the request that needs to write node values,

  14. ParematerValue -> ParameterValue
  15. "Case1: TR-069 data type is “base64”." current: b. Otherwise, then it constructs DmtData(int, byte[]) with the first elements in the array of raw format names as the first argument. should be: b. Otherwise, then it constructs DmtData(String formatName, byte[] data) with the first elements in the array of raw format names as the first argument.
  16. setting ParematerValue for the node is failed. -> Setting ParameterValue for the node is failed.
  17. , Otherwise go to Step 4 -> , otherwise go to Step 4
  18. , The CWMP Fault code specified -> , the CWMP Fault code specified
  19. IIf the PA encounters -> If the PA encounters
  20. " If the SOAP fault does not contain SetParameterValuesFault, the protocol adapter must set a code and a message contained in a DmtException instance to a FaultCode element and a FaultString element in the SOAP Fault response, respectively. The code in the DmtException must change the Fault code according to Table 5. Other information such as a path and causes included in the DmtException instance may be added to the FaultString element. " The paragraph looks duplicated with the next one.

  21. MetaData -> MetaNode
  22. current: FORMAT_STRING because FORMAT_XML will never be selected for SetParameterValues RPC (see 6.2.1.2) should be: FORMAT_STRING because FORMAT_XML will never be selected for SetParameterValues RPC (see 6.4.2, Case 3)

bjhargrave commented 13 years ago

Comment author: Ikuo Yamasaki <yamasaki.ikuo@lab.ntt.co.jp>

Although I had not checked all in detail, all except No.9 has no issue to be discussed. (Just I'll revise RFC149).

Regarding No.9, we need to discuss.

  1. Mapping of AddObject RPC to Dmt Admin interfaces, step 3 Step 3. If not, it is an error condition. - it's not clear how to handle that error condition.

How about sending back CWMP Fault with "9002 Internal error" or "9005 Invalid parameter name" ?

bjhargrave commented 13 years ago

Comment author: Ikuo Yamasaki <yamasaki.ikuo@lab.ntt.co.jp>

REG F2F, Feb 14th, 2011: PA should manage mapping table for the data model imcompatible to TR-106. Therefore, PA should not assume it as an error. Ikuo will investigate it and update the RFC149.

bjhargrave commented 13 years ago

Comment author: Ikuo Yamasaki <yamasaki.ikuo@lab.ntt.co.jp>

I had updated RFC149 (ver 1.13.0). Evgeni, please close this bug if you confirm, especially for No9.

bjhargrave commented 13 years ago

Comment author: Evgeni Grigorov <e.grigorov@prosyst.com>

The document was revised in respect of fix confirmation. The following issues are still available:

  1. ParematerValue -> ParameterValue
  2. , Otherwise go to Step 4 -> , otherwise go to Step 4
  3. , The CWMP Fault code specified -> , the CWMP Fault code specified They are technical and will be fixed in spec doc. So, the issue is fixed.