Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: install server_sent_event_stage #2117

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

sloanelybutsurely
Copy link
Contributor

Installs server_sent_event_stage which requires also adding castore with explicitly to the mix.exs and specifying override: true.

server_sent_event_stage expects {:castore, "~> 0.1"} but castore 1.0.8 is already installed by another dependency. We can remove the override: true once server_sent_event_stage has been updated to support newer versions of castore.

The move to from 0.1.x to 1.0.x versions in castore was a non-breaking change1. Subsequent patch versions of castore are simply updates to the certificates included (removing old/expired certs and adding new ones).

Footnotes

  1. castore 0.1.22..1.0.0: https://diff.hex.pm/diff/castore/0.1.22..1.0.0

Installs server_sent_event_stage which requires also adding `castore`
with explicitly to the `mix.exs` and specifying `override: true`.

server_sent_event_stage expects `{:castore, "~> 0.1"}` but castore
1.0.8 is already installed by another dependency. We can remove the
`override: true` once server_sent_event_stage has been updated to
support newer versions of castore.

The move to from `0.1.x` to `1.0.x` versions in castore was a
non-breaking change[^1]. Subsequent patch versions of castore are simply
updates to the certificates included (removing old/expired certs and
adding new ones).


[^1]: castore 0.1.22..1.0.0: https://diff.hex.pm/diff/castore/0.1.22..1.0.0
@sloanelybutsurely sloanelybutsurely requested a review from a team as a code owner July 31, 2024 17:18
Copy link

Coverage of commit 2dc836d

Summary coverage rate:
  lines......: 45.1% (2956 of 6551 lines)
  functions..: 40.3% (1054 of 2616 functions)
  branches...: no data found

Files changed coverage rate: n/a

Download coverage report

@sloanelybutsurely sloanelybutsurely merged commit cf3e212 into main Aug 1, 2024
13 checks passed
@sloanelybutsurely sloanelybutsurely deleted the sloane/push-omxsssvqumrm branch August 1, 2024 11:08
@digitalcora
Copy link
Contributor

Since server_sent_event_stage is ours (...comrade), I went ahead and updated it. In the next release, we won't have to override ca_store.

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