Skip to content
This repository was archived by the owner on Apr 18, 2026. It is now read-only.

Docs: Add quickstart user guide#265

Merged
mlevesquedion merged 9 commits into
google:masterfrom
mlevesquedion:add-quickstart-user-guide
Feb 3, 2021
Merged

Docs: Add quickstart user guide#265
mlevesquedion merged 9 commits into
google:masterfrom
mlevesquedion:add-quickstart-user-guide

Conversation

@mlevesquedion
Copy link
Copy Markdown
Contributor

@mlevesquedion mlevesquedion commented Jan 28, 2021

This PR adds a user guide that shows how to configure and run the analyzer. The intent is to provide a "quickstart" tutorial that allows users to quickly grasp how they can start applying the analyzer.

  • (N/A) [ ] Running against a large codebase such as Kubernetes does not error out. (See DEVELOPING.md for instructions on how to do that.)
  • Appropriate changes to README are included in PR

@mlevesquedion mlevesquedion force-pushed the add-quickstart-user-guide branch from a263cd3 to b2a7231 Compare January 28, 2021 16:58
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md Outdated
After making this change, the analyzer no longer produces a report. There are many ways to
address reports, the important thing is to modify the code so that the Source can no longer
reach the Sink. For example, you may wish to define a "Sanitizer" function that removes the
password, making the `auth` struct safe to log. See [the documentation](../configuration/README.md)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Authentication

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Are you referring to auth? I think I prefer auth here, because that is IMO the most specific label, being the name of the variable in the example. WDYT?

Copy link
Copy Markdown
Member

@vinayakankugoyal vinayakankugoyal Jan 28, 2021

Choose a reason for hiding this comment

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

It says 'making the auth struct safe to log.' But there is no auth struct there is an Authentication struct.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

In my understanding, the auth variable is a struct of type Authentication. I see your point, though. The sanitizer would work with values of type Authentication.

I've reworded it. WDYT?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am thinking do we really need the line
For example, you may wish to define ...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think it's good to make a nod to it here. Sources, Sinks and Sanitizers are (in my understanding) the 3 core concepts to understand for levee. Sources and Sinks are more fundamental, which is why they get most of the attention in this guide. But I think Sanitizers should at least get a mention.

Comment thread guides/quickstart.md
Copy link
Copy Markdown
Contributor

@PurelyApplied PurelyApplied left a comment

Choose a reason for hiding this comment

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

Looks pretty good. I've got a handful of nits, but this is a good addition.

Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md Outdated
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md
Comment thread guides/quickstart.md Outdated
@mlevesquedion
Copy link
Copy Markdown
Contributor Author

Thank you for your comments! PTAAL.

@mlevesquedion
Copy link
Copy Markdown
Contributor Author

I've changed the config to use Value instead of Val when configuring the field tag, as per discussion here: #266 (comment)

I'm marking this PR a draft until that's merged. (Currently, using Value causes the configuration to be invalid, i.e. the tagged field won't be detected.)

@mlevesquedion mlevesquedion marked this pull request as draft February 1, 2021 14:20
@mlevesquedion mlevesquedion marked this pull request as ready for review February 1, 2021 20:56
@mlevesquedion mlevesquedion merged commit 2436ffd into google:master Feb 3, 2021
@mlevesquedion mlevesquedion deleted the add-quickstart-user-guide branch February 3, 2021 20:11
@mlevesquedion mlevesquedion changed the title Add quickstart user guide Docs: Add quickstart user guide Feb 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants