Uploaded image for project: 'Falcon'
  1. Falcon
  2. FALCON-1477

Adding "-debug" option to Falcon CLI for debug statements to stdout.

Details

    • New Feature
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.8
    • None
    • None

    Description

      With this feature, users can specify "-debug" in CLI arguments to get the debug statements in Stdout.
      eg. falcon entity -type feed -name in-feed.xml -submit -debug
      Also, users can export the env variable FALCON_DEBUG to a non-zero value to get the debug statements on stdout.
      Starting right now with getting basic debug output such as the Http connection URL to the server, this can be later extended for printing more debug statements at various stages.

      Attachments

        1. FALCON-1477-v0.patch
          13 kB
          Narayan Periwal
        2. FALCON-1477-v1.patch
          15 kB
          Narayan Periwal
        3. FALCON-1477-v2.patch
          17 kB
          Narayan Periwal

        Issue Links

          Activity

            Can checkIfSuccessful() print the jersey response object in debug mode ?

            sriksun Srikanth Sundarrajan added a comment - Can checkIfSuccessful() print the jersey response object in debug mode ?

            sriksun, we can print the jersey response object in debug mode in the method checkIfSuccessful().
            In some places, we dont have this checkIfSuccessful() method. In such cases, we will have to call the method printClientResponse() to print the jersey response object.

            nperiwal Narayan Periwal added a comment - sriksun , we can print the jersey response object in debug mode in the method checkIfSuccessful(). In some places, we dont have this checkIfSuccessful() method. In such cases, we will have to call the method printClientResponse() to print the jersey response object.

            sriksun, IMO either we should print the jersey object in printClientResponse() at all places, or if we print in checkIfSuccessful(), then we need to add the call at places where it is missing. This will make the solution clean.

            nperiwal Narayan Periwal added a comment - sriksun , IMO either we should print the jersey object in printClientResponse() at all places, or if we print in checkIfSuccessful(), then we need to add the call at places where it is missing. This will make the solution clean.

            nperiwal Thanks for working on this. Documentation to enable debugging is missing in the patch. Can you please add that as well.

            peeyushb Peeyush Bishnoi added a comment - nperiwal Thanks for working on this. Documentation to enable debugging is missing in the patch. Can you please add that as well.

            peeyushb Thanks for the suggestion. I have added the documentation to enable the debugging.

            nperiwal Narayan Periwal added a comment - peeyushb Thanks for the suggestion. I have added the documentation to enable the debugging.
            sowmyaramesh Sowmya Ramesh added a comment -

            nperiwal: Patch doesn't apply cleanly on latest code. Can you please rebase and attach the patch?!

            sowmyaramesh Sowmya Ramesh added a comment - nperiwal : Patch doesn't apply cleanly on latest code. Can you please rebase and attach the patch?!

            sowmyaramesh, Rebased the patch, uploaded after testing the functionality.

            nperiwal Narayan Periwal added a comment - sowmyaramesh , Rebased the patch, uploaded after testing the functionality.
            pallavi.rao Pallavi Rao added a comment -

            Committed patch to master : 4bc3898
            nperiwal, thank you for your contribution.

            pallavi.rao Pallavi Rao added a comment - Committed patch to master : 4bc3898 nperiwal , thank you for your contribution.

            People

              nperiwal Narayan Periwal
              nperiwal Narayan Periwal
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: