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

rocket_contrib conflicts with latest ring (0.14) #905

Closed
kpcyrd opened this issue Jan 29, 2019 · 10 comments
Closed

rocket_contrib conflicts with latest ring (0.14) #905

kpcyrd opened this issue Jan 29, 2019 · 10 comments
Assignees
Labels
accepted An accepted request or suggestion deficiency Something doesn't work as well as it could

Comments

@kpcyrd
Copy link

kpcyrd commented Jan 29, 2019

I'm currently trying to prepare a bugfix release but I'm having trouble due to the ring dependency in the cookie crate. It's currently depending on an outdated ring version, but since I don't need cookies at all I would like to disable the need for ring.

It's already possible to build rocket without ring like this:

rocket = { version = "0.4", default-features = false }

This disables the private-cookies feature, which in turn disables the private-cookies feature in rocket_http, which disables the secure feature in cookie.

private-cookies = ["rocket_http/private-cookies"]

This works fine unless you also need rocket_contrib, which is the case for me. rocket_contrib depends on rocket with default features and no way to disable it, which means that the ring dependency in cookie is getting enabled again:

rocket = { version = "0.4.0", path = "../../core/lib/" }

Introducing a feature to disable ring in rocket_contrib would be the best solution for my problem, I'm currently trying to find an alternative solution since that specific version is faulty on aarch64.

Thanks!

@SuperHacker-liuan
Copy link

Confirmed. I had to use a git version of rocket contrib to fix the ring compile problem. Actually, private cookies is needn't by rocket_contrib.

@jebrosen jebrosen added accepted An accepted request or suggestion deficiency Something doesn't work as well as it could labels Jan 30, 2019
@jebrosen
Copy link
Collaborator

It makes sense for rocket_contrib to only enable private cookies if it needs to and is probably doable as a small change in a minor release. If any features do need private-cookies it can be enabled on those features individually.

@jebrosen
Copy link
Collaborator

jebrosen commented Feb 2, 2019

Resolved in 90296dd in master.

@jebrosen jebrosen closed this as completed Feb 2, 2019
@kpcyrd
Copy link
Author

kpcyrd commented Feb 2, 2019

@SergioBenitez any updates regarding a 0.4.1 release?

@SergioBenitez
Copy link
Member

@kpcyrd Yes. I'm going through all issues and PRs now in hopes of getting out a new release in the next few days.

@kpcyrd
Copy link
Author

kpcyrd commented Feb 19, 2019

Any news on this one?

@kpcyrd
Copy link
Author

kpcyrd commented Mar 4, 2019

I've pushed my first release with a workaround in the meantime to address the AES-GCM issue on aarch64 (briansmith/ring#628).

I've added this to my workspace:

[patch.crates-io]
rocket = { git = "https://github.com/SergioBenitez/Rocket.git", rev="c86f4312fb273c0380fb76a97bfb7fc227800542" }
rocket_contrib = { git = "https://github.com/SergioBenitez/Rocket.git", rev="c86f4312fb273c0380fb76a97bfb7fc227800542" }

During the release I disable this block, remove the server from the workspace, regenerate Cargo.lock and push a release. Afterwards I re-add the server to the workspace, re-enable the [patch] section and regenerate Cargo.lock. Building the rocket server from release tarballs isn't supported anymore in my project.

It seems this issue is also related to #650.

@kpcyrd
Copy link
Author

kpcyrd commented Mar 25, 2019

@SergioBenitez any news?

@kpcyrd
Copy link
Author

kpcyrd commented Apr 14, 2019

@SergioBenitez @jebrosen Any news here? I'm considering migrating away from rocket at this point.

@kpcyrd kpcyrd changed the title rocket_contrib is missing private-cookies flag rocket_contrib conflicts with latest ring (0.14) Apr 22, 2019
kpcyrd added a commit to kpcyrd/sn0int that referenced this issue Apr 24, 2019
@SergioBenitez
Copy link
Member

@kpcyrd The release happened a couple of days ago, finally. Apologies for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted An accepted request or suggestion deficiency Something doesn't work as well as it could
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants