Skip to content

Doc: Clean up JA3 plugin docs.#5454

Merged
SolidWallOfCode merged 1 commit intoapache:masterfrom
SolidWallOfCode:doc-ja3-plugin-fix
May 16, 2019
Merged

Doc: Clean up JA3 plugin docs.#5454
SolidWallOfCode merged 1 commit intoapache:masterfrom
SolidWallOfCode:doc-ja3-plugin-fix

Conversation

@SolidWallOfCode
Copy link
Copy Markdown
Member

No description provided.

dyrock
dyrock previously approved these changes May 9, 2019
Copy link
Copy Markdown
Contributor

@dyrock dyrock left a comment

Choose a reason for hiding this comment

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

+1 fixes my poor documentation

Comment thread doc/admin-guide/plugins/ja3_fingerprint.en.rst Outdated
Comment thread doc/admin-guide/plugins/ja3_fingerprint.en.rst Outdated
@bryancall
Copy link
Copy Markdown
Contributor

[approve ci doc]

@bryancall
Copy link
Copy Markdown
Contributor

[approve ci docs]

1 similar comment
@randall
Copy link
Copy Markdown
Contributor

randall commented May 10, 2019

[approve ci docs]

@SolidWallOfCode SolidWallOfCode force-pushed the doc-ja3-plugin-fix branch 4 times, most recently from bc17f1c to 18c31cd Compare May 13, 2019 19:59
Copy link
Copy Markdown
Contributor

@dyrock dyrock left a comment

Choose a reason for hiding this comment

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

I think there should be no space between finger and print.

Copy link
Copy Markdown
Contributor

@dyrock dyrock left a comment

Choose a reason for hiding this comment

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

looks good to me!

@SolidWallOfCode SolidWallOfCode merged commit 04543ec into apache:master May 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants