Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for creating private CTF channels #224

Conversation

l4cr0ss
Copy link
Contributor

@l4cr0ss l4cr0ss commented Jun 4, 2020

Fix issue #125

Allow creation of a private CTF channel via the -private flag, i.e. !addctf -private defcon2020

Add util method for parsing args for options.

  • This method takes an array and searches for any strings in that array beginning with a hyphen.
  • If it encounters such a string it strips the leading punctuation from that string and puts it into a new array.
  • After iterating the given array it returns a 2-tuple containing the new array of options sans leading punctuation and the original array, minus any extracted options.

Fix issue OpenToAllCTF#125

Add util method for parsing args for options. This method takes an array and
searches for any strings in that array beginning with a hyphen. If it
encounters such a string it strips the leading punctuation from that string and
puts it into a new array. After iterating the given array it returns a 2-tuple
containing the new array of options sans leading punctuation and the original
array minus those extracted options.
@l4cr0ss l4cr0ss changed the title Add support for creating private CTF channels [WIP] Add support for creating private CTF channels Jun 4, 2020
@l4cr0ss l4cr0ss changed the title [WIP] Add support for creating private CTF channels (WIP) Add support for creating private CTF channels Jun 4, 2020
@l4cr0ss l4cr0ss marked this pull request as draft June 4, 2020 19:25
@l4cr0ss l4cr0ss changed the title (WIP) Add support for creating private CTF channels Add support for creating private CTF channels Jun 4, 2020
opts.append(arg)
else:
args.append(arg)
return (opts,args)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

outer parens are superfluous

@l4cr0ss
Copy link
Contributor Author

l4cr0ss commented Jan 19, 2021

This will go in as part of #233 via a config flag, closing.

@l4cr0ss l4cr0ss closed this Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants