Skip to content

Ensure only exact name matches are added to the index #1550

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

Merged

Conversation

jtgeibel
Copy link
Member

@jtgeibel jtgeibel commented Nov 4, 2018

On the client side cargo checks dependency names against the index and
requires an exact match. For instance, a user cannot depend on
diesel-migrations or Diesel_migrations as cargo generates an error
directing them to use the correct name diesel_migrations.

This check for an exact name match is now repeated on the server when a
crate is published. This ensures that bugs in cargo or third-party
tooling cannot result in a dependency name that cargo is unable to
locate in the index.

On the client side cargo checks dependency names against the index and
requires an exact match.  For instance, a user cannot depend on
`diesel-migrations` or `Diesel_migrations` as cargo generates an error
directing them to use the correct name `diesel_migrations`.

This check for an exact name match is now repeated on the server when a
crate is published.  This ensures that bugs in cargo or third-party
tooling cannot result in a dependency name that cargo is unable to
locate in the index.
@sgrif
Copy link
Contributor

sgrif commented Nov 5, 2018

bors: r+

bors-voyager bot added a commit that referenced this pull request Nov 5, 2018
1550: Ensure only exact name matches are added to the index r=sgrif a=jtgeibel

On the client side cargo checks dependency names against the index and
requires an exact match.  For instance, a user cannot depend on
`diesel-migrations` or `Diesel_migrations` as cargo generates an error
directing them to use the correct name `diesel_migrations`.

This check for an exact name match is now repeated on the server when a
crate is published.  This ensures that bugs in cargo or third-party
tooling cannot result in a dependency name that cargo is unable to
locate in the index.

Co-authored-by: Justin Geibel <[email protected]>
@bors-voyager
Copy link
Contributor

bors-voyager bot commented Nov 5, 2018

Build succeeded

@bors-voyager bors-voyager bot merged commit 86dffda into rust-lang:master Nov 5, 2018
@jtgeibel jtgeibel deleted the require-dependency-name-exact-match branch February 7, 2019 16:14
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.

2 participants