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

Update the plug_crypto dependency to version 2.0.0 #193

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

omarelb
Copy link

@omarelb omarelb commented Dec 24, 2023

The outdated dependency was causing my project to be unable to compile.

Copy link

@peaceful-james peaceful-james left a comment

Choose a reason for hiding this comment

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

I am looking forward to this being merged

@sgerrand
Copy link
Contributor

👋 @alan @msz: Are you able to trigger the test suite checks for @omarelb? I'm assuming that it hasn't run because this is their first contribution to this project.

@kaylenmistry
Copy link
Contributor

+1

1 similar comment
@dudeman
Copy link

dudeman commented Feb 9, 2024

+1

@astorre88
Copy link

Hey guys! Just curious when it's planned to be merged?)

@Cellane
Copy link

Cellane commented Jun 27, 2024

Hello! Any chance of moving forward with this? Would love to adopt this library in our project but this is preventing us from doing so.

@edennis
Copy link

edennis commented Sep 13, 2024

Any chance of getting this change or similar in (plug_crypto is now at 2.1)?

@peaceful-james
Copy link

I have switched to using :flop (https://github.com/woylie/flop) because this repo seems abandoned.

@Cellane
Copy link

Cellane commented Sep 14, 2024

We have switched to Flop just last week as well 👌

@@ -40,7 +40,7 @@ defmodule Paginator.Mixfile do
{:ex_machina, "~> 2.1", only: :test},
{:inch_ex, "~> 2.0", only: [:dev, :test]},
{:postgrex, "~> 0.13", optional: true},
{:plug_crypto, "~> 1.2.0"}
{:plug_crypto, "~> 2.0.0"}

Choose a reason for hiding this comment

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

I think, we could make it more permissive.

Suggested change
{:plug_crypto, "~> 2.0.0"}
{:plug_crypto, "~> 1.2.0 or ~> 2.0"}

As far as I can see in plug_crypto changelog https://github.com/elixir-plug/plug_crypto/blob/main/CHANGELOG.md

The API did not change between the two. The major version got bumped because of the internal algorithm change. However, the transition is smooth.

That means, it can work with both 1.2.x and 2.x.x.
We still keep the 1.2.0 minimum because it fixed an important bug.

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.