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

Still maintained? #81

Open
smessmer opened this issue Mar 8, 2022 · 5 comments
Open

Still maintained? #81

smessmer opened this issue Mar 8, 2022 · 5 comments

Comments

@smessmer
Copy link

smessmer commented Mar 8, 2022

Is this still maintained? I see PRs with fixes going back to 2017 but nothing seems to get merged.

@smessmer smessmer changed the title Still maintaned? Still maintained? Apr 11, 2022
@pinkforest
Copy link

pinkforest commented Jul 30, 2022

Hi @Kimundi @avitex @NikVolf

Would you be interested on handing over maintenance over or commit to fixing some soundness issues ?

The crate has ~ 12 M downloads as of today and it would be nice to fix these and keep it maintained

We are thinking of raising advisories here
rustsec/advisory-db#1174

The unsoundness has been documented here
https://github.com/noamtashma/owning-ref-unsoundness

Soudness Issues
#49
#61
#71
#77

@utilForever
Copy link

I created a repository that fixes some soundness issues.
https://github.com/utilForever/owning-ref-rs

@Ten0
Copy link

Ten0 commented Aug 14, 2022

That would be really really useful considering how recurring this pattern is.
Currently I refrain from using it because of the footguns implied by the unsoundness, but that would otherwise become a must!

@Ten0
Copy link

Ten0 commented Aug 14, 2022

I created a repository that fixes some soundness issues. https://github.com/utilForever/owning-ref-rs

With the history being entirely rewritten it's unclear what unsoundness issues are fixed/to what extent these differ from the open PRs.

@pinkforest
Copy link

pinkforest commented Aug 14, 2022

prometheus-client addressed this by using parking_lot here: prometheus/client_rust#78

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

No branches or pull requests

4 participants