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

Already on GitHub? Sign in to your account

XML Report bug in issue name #263

Closed
sunilkr opened this Issue Aug 7, 2012 · 15 comments

Comments

Projects
None yet
2 participants

sunilkr commented Aug 7, 2012

There is a bug in XML report generation.
Value of the element /arachni_report/issues/issue/name should have been taken from Module#info['name'](the way it used to be).
But now it is being taken from Module#info['issue']['name'].

sunilkr commented Aug 7, 2012

The bug seems specific to 'recon' modules.

Owner

Zapotek commented Aug 7, 2012

Issues are being initialized in a stupid, stupid manner -- very early implementation mistake.
The Issue logging system is scheduled for a massive overhaul to fix it -- see #209.

For the time being, I'll sort this out in whatever way possible.

@ghost ghost assigned Zapotek Aug 7, 2012

Owner

Zapotek commented Aug 7, 2012

Ah, I read your initial description the other way around; that's no bug, that's the way it should be.

issue/name = the name of the issue (i.e. Module#info[:issue][:name)
issue/mod_name = name of the module that logged the issue (i.e. Module#info[:name])

@Zapotek Zapotek closed this Aug 7, 2012

sunilkr commented Aug 8, 2012

Well It does not happen with other modules.
I guess the Module#info[:issue][:name] should be in issues/issue/variation.
I can send you the XML if you want.

On Wed, Aug 8, 2012 at 3:16 AM, Tasos Laskos notifications@github.comwrote:

Ah, I read your initial description the other way around; that's no bug,
that's the way it should be.

issue/name = the name of the issue (i.e. Module#info[:issue][:name)
issue/mod_name = name of the module that logged the issue (i.e.
Module#info[:name])


Reply to this email directly or view it on GitHubhttps://github.com/Arachni/arachni/issues/263#issuecomment-7567947.

Your smile is the most precious thing that doesn't cost you. Keep smiling.

:)

 SunilKumar

http://in.linkedin.com/in/sunilkr86/

Owner

Zapotek commented Aug 8, 2012

Yes please do because I couldn't reproduce this.

sunilkr commented Aug 9, 2012

issue #6.

Everything else is taken from Module#info except name which is from
Module#info[:issue].
The content of the looks more like part of description. Not many
modules follow this pattern.

On Wed, Aug 8, 2012 at 7:30 PM, Tasos Laskos notifications@github.comwrote:

Yes please do because I couldn't reproduce this.


Reply to this email directly or view it on GitHubhttps://github.com/Arachni/arachni/issues/263#issuecomment-7584290.

Your smile is the most precious thing that doesn't cost you. Keep smiling.

:)

 SunilKumar

http://in.linkedin.com/in/sunilkr86/

Owner

Zapotek commented Aug 9, 2012

Could I see the report?

sunilkr commented Aug 9, 2012

The XML is in attachment.

On Thu, Aug 9, 2012 at 7:07 PM, Tasos Laskos notifications@github.comwrote:

Could I see the report?


Reply to this email directly or view it on GitHubhttps://github.com/Arachni/arachni/issues/263#issuecomment-7614187.

Your smile is the most precious thing that doesn't cost you. Keep smiling.

:)

 SunilKumar

http://in.linkedin.com/in/sunilkr86/

sunilkr commented Aug 9, 2012

oooops. I sent the attachment with email reply which seems to be stripped here.

Owner

Zapotek commented Aug 9, 2012

Great, got it now thanks.

@Zapotek Zapotek reopened this Aug 15, 2012

Owner

Zapotek commented Aug 15, 2012

AFR report was corrupted.

Owner

Zapotek commented Aug 22, 2012

Could you give it another shot please and send me the report again?
Better compress it first to avoid any corruption during transport.

Owner

Zapotek commented Sep 10, 2012

@sunilkr ping

sunilkr commented Sep 10, 2012

I can't test that now, sorry.

On Mon, Sep 10, 2012 at 7:28 AM, Tasos Laskos notifications@github.comwrote:

@sunilkr https://github.com/sunilkr ping


Reply to this email directly or view it on GitHubhttps://github.com/Arachni/arachni/issues/263#issuecomment-8411959.

Your smile is the most precious thing that doesn't cost you. Keep smiling.

:)

 SunilKumar

http://in.linkedin.com/in/sunilkr86/

Owner

Zapotek commented Sep 10, 2012

Can't repro.

@Zapotek Zapotek closed this Sep 10, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment