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 bugreport cmd #533

Merged
merged 8 commits into from Sep 30, 2018

Conversation

Projects
None yet
4 participants
@GrosQuildu
Contributor

GrosQuildu commented Sep 25, 2018

for #414
Not sure we need it since there is version() command (this adds session history, platform and optional browser opening)

@disconnect3d

This comment has been minimized.

Show comment
Hide comment
@disconnect3d

disconnect3d Sep 25, 2018

Member

Can you show it in action? e.g. with asciinema

Member

disconnect3d commented Sep 25, 2018

Can you show it in action? e.g. with asciinema

@GrosQuildu

This comment has been minimized.

Show comment
Hide comment
@GrosQuildu

GrosQuildu Sep 25, 2018

Contributor

https://asciinema.org/a/8rAtszQaH80Zzx3KTIm55is1j

travis failed, is it supposed to fail? :D

Contributor

GrosQuildu commented Sep 25, 2018

https://asciinema.org/a/8rAtszQaH80Zzx3KTIm55is1j

travis failed, is it supposed to fail? :D

Show outdated Hide outdated pwndbg/commands/version.py Outdated
Show outdated Hide outdated pwndbg/commands/version.py Outdated
@anthraxx

This comment has been minimized.

Show comment
Hide comment
@anthraxx

anthraxx Sep 25, 2018

Contributor

awesome, its green 👍

Contributor

anthraxx commented Sep 25, 2018

awesome, its green 👍

@disconnect3d

This comment has been minimized.

Show comment
Hide comment
@disconnect3d

disconnect3d Sep 26, 2018

Member

Can we also add info about that bugreport command to the exception handler, i.e. that he can fire one command and a browser with filled bugreport will spawn for them?

Member

disconnect3d commented Sep 26, 2018

Can we also add info about that bugreport command to the exception handler, i.e. that he can fire one command and a browser with filled bugreport will spawn for them?

@disconnect3d

See comments

@ecx86

This comment has been minimized.

Show comment
Hide comment
@ecx86

ecx86 Sep 29, 2018

Contributor

I'm not sure about opening the browser. 99% of the time I use gdb, it's on a headless box over SSH.

Contributor

ecx86 commented Sep 29, 2018

I'm not sure about opening the browser. 99% of the time I use gdb, it's on a headless box over SSH.

@GrosQuildu

This comment has been minimized.

Show comment
Hide comment
@GrosQuildu

GrosQuildu Sep 30, 2018

Contributor

Noidea, whatever you propose. But it's optional only.

Contributor

GrosQuildu commented Sep 30, 2018

Noidea, whatever you propose. But it's optional only.

@disconnect3d

This comment has been minimized.

Show comment
Hide comment
@disconnect3d

disconnect3d Sep 30, 2018

Member

Hmm.. maybe lets change the flag from --browse to --run-browser so it will be more explicit that it is optional.

Member

disconnect3d commented Sep 30, 2018

Hmm.. maybe lets change the flag from --browse to --run-browser so it will be more explicit that it is optional.

GrosQuildu and others added some commits Sep 30, 2018

@disconnect3d disconnect3d merged commit b847f1d into pwndbg:dev Sep 30, 2018

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment