_output_ should strip trailing forward slashes #56

Activity

1. codingo

   added

   bugSomething isn't workingSomething isn't working

   good first issueGood for newcomersGood for newcomers

   on Aug 15, 2019
2. prodigysml commented on Aug 15, 2019

   

   prodigysml

   on Aug 15, 2019 via email

   Collaborator

   More actions

   Good thinking there! Didn't realise that. Definitely worth doing.

   …

   On Thu, 15 Aug. 2019, 10:56 pm Michael Skelton, ***@***.***> wrote: When building command files I notice that the trend is to build them as: nmap -sC -sV -oA _output_/scans/nmap _target_ Noting that a slash after _output_ is the trend. The problem with this is that it's not uncommon to then launch this command file as something like: interlace -cL example.il -t 127.0.0.1 -o ./ This results in a path that has // within it. To better support such use cases, as a default the output flag should strip any trailing forward slashes. This should also be noted within README.md as expected behavior. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#56?email_source=notifications&email_token=AEBVTU62HL4UHTVGCH5GYXLQEVG6DA5CNFSM4IL56WY2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFNSXDA>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEBVTU44KGOWX5H3XJ7IDELQEVG6DANCNFSM4IL56WYQ> .

   👍React with 👍1Reacted by Michael Skelton
3. codingo commented on Aug 16, 2019

   

   codingo

   on Aug 16, 2019

   OwnerAuthor

   More actions

   Closed with #57
4. codingo

   closed this as completedon Aug 16, 2019