Skip to content

add invocation to README#9

Merged
pongad merged 4 commits intomasterfrom
pongad-patch-1
Sep 25, 2018
Merged

add invocation to README#9
pongad merged 4 commits intomasterfrom
pongad-patch-1

Conversation

@pongad
Copy link
Copy Markdown
Contributor

@pongad pongad commented Sep 21, 2018

No description provided.

@pongad pongad requested a review from jba September 21, 2018 19:20
Comment thread README.md Outdated
`protoc --go_gapic_out [OUTPUT_DIR] --go_gapic_opt 'package/path/url;name' a.proto b.proto`

The `go_gapic_opt` flag is necessary because we need to know where to generated file will live.
The substring before the semicolon is the import path of the package eg `github.com/username/awesomeness`.

This comment was marked as spam.

This comment was marked as spam.

Comment thread README.md Outdated
The `go_gapic_opt` flag is necessary because we need to know where to generated file will live.
The substring before the semicolon is the import path of the package eg `github.com/username/awesomeness`.
The substring after the semicolon is the name of the package used in the `package` statement.
Idiomatically the name is last element of the path but it needs not be, eg if the last element of the path is a version.

This comment was marked as spam.

This comment was marked as spam.

@pongad
Copy link
Copy Markdown
Contributor Author

pongad commented Sep 21, 2018

@jba PTAL

Comment thread README.md Outdated
The `go_gapic_opt` flag is necessary because we need to know where to generated file will live.
The substring before the semicolon is the import path of the package, e.g. `github.com/username/awesomeness`.
The substring after the semicolon is the name of the package used in the `package` statement.
Idiomatically the name is last element of the path but it needs not be.

This comment was marked as spam.

TIL: Modal auxiliary verbs do not change form with the subject.
@pongad pongad merged commit 2f819e7 into master Sep 25, 2018
@pongad pongad deleted the pongad-patch-1 branch September 25, 2018 17:59
julieqiu added a commit to julieqiu/gapic-generator-go that referenced this pull request Oct 5, 2023
Vulnerability #1: GO-2023-1571
    Denial of service via crafted HTTP/2 stream in net/http and golang.org/x/net
  More info: https://pkg.go.dev/vuln/GO-2023-1571
  Module: golang.org/x/net
    Found in: golang.org/x/net@v0.0.0-20210405180319-a5a99cb37ef4
    Fixed in: golang.org/x/net@v0.7.0
    Example traces found:
      #1: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.ErrCode.String
      #2: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.FrameType.String
      #3: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.Setting.String
      #4: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.SettingID.String
      googleapis#5: internal/printer/printer.go:62:13: printer.P.Printf calls fmt.Fprintf, which eventually calls http2.chunkWriter.Write
      googleapis#6: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.duplicatePseudoHeaderError.Error
      googleapis#7: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.headerFieldNameError.Error
      googleapis#8: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.headerFieldValueError.Error
      googleapis#9: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.pseudoHeaderError.Error
      googleapis#10: internal/gengapic/options.go:172:21: gengapic.transport.String calls fmt.Sprintf, which eventually calls http2.writeData.String
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants