Skip to content

deps: update libcurl to 7.77.0#16676

Merged
wrowe merged 1 commit intoenvoyproxy:mainfrom
tonya11en:curl_update
May 26, 2021
Merged

deps: update libcurl to 7.77.0#16676
wrowe merged 1 commit intoenvoyproxy:mainfrom
tonya11en:curl_update

Conversation

@tonya11en
Copy link
Copy Markdown
Member

Picking up libcurl, with the latest security bugfixes. Find the libcurl 7.77.0 release notes here.

Signed-off-by: Tony Allen <tony@allen.gg>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label May 26, 2021
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #16676 was opened by tonya11en.

see: more, trace.

@moderation
Copy link
Copy Markdown
Contributor

Curl - the gift that keeps on giving. Bumping this deps normally breaks the patch - https://github.com/envoyproxy/envoy/blob/main/bazel/foreign_cc/curl.patch - but maybe we will get lucky

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label May 26, 2021
@wrowe
Copy link
Copy Markdown
Contributor

wrowe commented May 26, 2021

The patch did not break...

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16676 (comment) was created by @wrowe.

see: more, trace.

Copy link
Copy Markdown
Contributor

@wrowe wrowe left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@wrowe wrowe merged commit 6ada505 into envoyproxy:main May 26, 2021
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: Tony Allen <tony@allen.gg>
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.

3 participants