Skip to content

properly read LRO type from proto annotation#10

Merged
pongad merged 4 commits intogoogleapis:masterfrom
pongad:lro-name
Sep 25, 2018
Merged

properly read LRO type from proto annotation#10
pongad merged 4 commits intogoogleapis:masterfrom
pongad:lro-name

Conversation

@pongad
Copy link
Copy Markdown
Contributor

@pongad pongad commented Sep 21, 2018

and proplery reset writer when testing!

and proplery reset writer when testing!
@pongad pongad requested a review from jba September 21, 2018 21:41
Comment thread internal/gengapic/lro.go Outdated
}

// Protoc adds '.' in front of names to signify they are fully qualified.
// Do that here so we can look up.

This comment was marked as spam.

This comment was marked as spam.

@pongad pongad merged commit e8246bd into googleapis:master Sep 25, 2018
@pongad pongad deleted the lro-name branch September 25, 2018 18:07
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