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 screenshot feature #8

Merged
merged 19 commits into from Dec 31, 2019
Merged

add screenshot feature #8

merged 19 commits into from Dec 31, 2019

Conversation

ghost
Copy link

@ghost ghost commented Dec 30, 2019

  • use chrome headless via proxy
  • create a screenshots dir [[username]]/[[websiteHost]].png

@tdh8316
Copy link
Owner

tdh8316 commented Dec 31, 2019

Hi! I appreciate your contribution!
I have trouble with using the -s option.

On Windows 10, Go 1.13.5 AMD64 Environment, I ran Investigo with this command:
go run Investigo.go tdh8316 -s

It opened many chrome browser windows. And when the process completed, these windows are not closed automatically. (The browser didn't connect to any site)
The screenshot files were not generated, despite the screenshot folder is located.

Could you let me know what your development environment is and how to use it?

@ghost
Copy link
Author

ghost commented Dec 31, 2019

Can you pull the latest commit and try again ? I enabled logs for the chrome screenshoter and let me know what it says. (ps. I don't have windows, can have one laptop by tomorrow)

@tdh8316 tdh8316 changed the base branch from master to develop December 31, 2019 14:54
@tdh8316 tdh8316 merged commit b4227df into tdh8316:develop Dec 31, 2019
tdh8316 added a commit that referenced this pull request Jan 4, 2020
* add go.mod files

* add screenshot function

* add chrome package, add screenshot option

* enable logs for chrome

* add screenshots

* dockerized investigo, add docker-compose, add container for standalone and chromium mode

* add travis-ci badge

* add more badges, add docker documentation

* change align to center

* fix badges

* add docker container for chromium

* fix README

* remove unecessary code

* fix README

* fix travis

* fix docker namespaces

* remove squash command
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.

1 participant