Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

dep: bump up caddy version to 2.7.3#55606

Merged
evict merged 6 commits into
mainfrom
vincent+will/update-caddy
Aug 16, 2023
Merged

dep: bump up caddy version to 2.7.3#55606
evict merged 6 commits into
mainfrom
vincent+will/update-caddy

Conversation

@evict

@evict evict commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

This PR makes sure we use a later version of Caddy, with less vulnerabilities. We move from version 2.4.5 to 2.7.3.

Test plan

Tested the changes locally using sg start, confirms it's still serving SSL with 2.7.3.

@evict evict requested a review from willdollman August 7, 2023 09:48
@cla-bot cla-bot Bot added the cla-signed label Aug 7, 2023
@evict evict requested a review from a team August 7, 2023 09:55
Comment thread dev/caddy.sh Outdated

@burmudar burmudar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

2.7.3 got released over the weekend and the official recommendation is to use that rather than 2.7.2

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>

@burmudar burmudar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See previous comment

Comment thread sg.config.yaml Outdated
Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>
@evict evict changed the title dep: bump up caddy version to 2.7.2 dep: bump up caddy version to 2.7.3 Aug 7, 2023
@willdollman

Copy link
Copy Markdown
Contributor

Waiting on caddy 2.7.3 docker images to be released

@willdollman

Copy link
Copy Markdown
Contributor

@willdollman willdollman enabled auto-merge (squash) August 15, 2023 10:20
@willdollman willdollman disabled auto-merge August 15, 2023 10:21
@evict evict enabled auto-merge (squash) August 16, 2023 09:42
@evict evict merged commit d13bc81 into main Aug 16, 2023
@evict evict deleted the vincent+will/update-caddy branch August 16, 2023 09:48
github-actions Bot pushed a commit that referenced this pull request Aug 16, 2023
* dep: bump up caddy version

* update changelog

* dep: use caddy 2.7.3

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>

* dep: use caddy 2.7.3

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>

---------

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>
Co-authored-by: Will Dollman <will.dollman@sourcegraph.com>
(cherry picked from commit d13bc81)
sanderginn pushed a commit that referenced this pull request Aug 16, 2023
dep: bump up caddy version to 2.7.3 (#55606)

* dep: bump up caddy version

* update changelog

* dep: use caddy 2.7.3

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>

* dep: use caddy 2.7.3

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>

---------

Co-authored-by: William Bezuidenhout <william.bezuidenhout@sourcegraph.com>
Co-authored-by: Will Dollman <will.dollman@sourcegraph.com>
(cherry picked from commit d13bc81)

Co-authored-by: Vincent <evict@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants