Uploaded image for project: 'controller'
  1. controller
  2. CONTROLLER-148

Controller should not allow to create flow with different flowID in URI and xml body

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Resolution: Done
    • None
    • None
    • mdsal
    • None
    • Operating System: Linux
      Platform: PC

    • 404
    • High

    Description

      I created a flow with different flowID in URI and xml body as stated below:

      PUT : http://192.168.56.1:8080/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/0/flow/24

      <?xml version="1.0" encoding="UTF-8" standalone="no"?>
      <flow xmlns="urn:opendaylight:flow:inventory">
      <priority>2</priority>
      <flow-name>Foo</flow-name>
      <match>
      <ethernet-match>
      <ethernet-type>
      <type>2048</type>
      </ethernet-type>
      </ethernet-match>
      <ipv4-destination>10.0.10.2/24</ipv4-destination>
      </match>
      <id>25</id>
      <table_id>1</table_id>
      <instructions>
      <instruction>
      <order>0</order>
      <apply-actions>
      <action>
      <order>0</order>
      <dec-nw-ttl/>
      </action>
      </apply-actions>
      </instruction>
      </instructions>
      </flow>

      The flow gets created and RESPONSE is 200 ok.

      When I hit the below URI it fails to generate the xml.
      GET : http://192.168.56.1:8080/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/0/flow/24

      On the other hand when I hit the below URI, response is 404 Not Found
      GET : http://192.168.56.1:8080/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/0/flow/25

      The controller should not allow to create a flow with diffrent flowId in URI and xml body at the first place itself.

      Thanks
      Faizan

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            vdemcak@cisco.com Vaclav Demcak
            md.faizan@tcs.com Md Faizan
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: