@klamontagne this change sounds like a good idea.

What do you think about simplifying this to do the environment lookup here in `main()`. That would avoid having to have 3 different variables for the same type of value 

```go
if *clientID != "" {
   *clientID = os.Getenv("google_auth_client_id")
}
```