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

C1fApp Initial version #63

Closed
wants to merge 2 commits into from
Closed

C1fApp Initial version #63

wants to merge 2 commits into from

Conversation

etz69
Copy link
Contributor

@etz69 etz69 commented Jun 20, 2017

C1fApp is a threat feed aggregator. This can be used to query any of the OSINT feeds

@saadkadhi saadkadhi mentioned this pull request Jun 20, 2017
@saadkadhi
Copy link
Contributor

Thanks @etz69 for your contribution. We do appreciate it! We will be reviewing it as soon as feasible.

While we have not written a policy for accepting submissions yet, please note that an issue must be created prior to making PRs. That way, we can ensure that multiple people won't waste their time working on the same analyzer. I've done it this time around 😉

Moreover, we do not accept aliases as an author name. Can you please provide your real name, eventually with an org name (not required)? We would like to give proper credits to the original author(s).

@etz69
Copy link
Contributor Author

etz69 commented Jun 21, 2017

Hi Saad , apologies for the omission of opening an issue and email. I have now updated my github profile ! Let me know if you want me to do the above updates and create a new pull request

@saadkadhi
Copy link
Contributor

Yes please, update the author and resubmit. Thanks.

@etz69
Copy link
Contributor Author

etz69 commented Jul 31, 2017

C1fApp is a threat feed aggregator. This can be used to query any of the OSINT feeds

@dlambrou
Copy link

addresses issue #103

@saadkadhi
Copy link
Contributor

Awesome, thanks! @etz69 can you rebase your PR against develop please?

@etz69 etz69 closed this Oct 25, 2017
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.

3 participants