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

"unauthorised releases" aren't flagged by MetaCPAN #706

Closed
neilb opened this issue Nov 16, 2012 · 1 comment
Closed

"unauthorised releases" aren't flagged by MetaCPAN #706

neilb opened this issue Nov 16, 2012 · 1 comment

Comments

@neilb
Copy link
Contributor

neilb commented Nov 16, 2012

If you upload a dist which contains a module for which you don't have permissions (i.e. the module is listed in $CPAN/modules/06perms.txt with someone else having permissions, but not you), then the upload will succeed, but the indexing will not.

search.cpan.org flags such releases. For example, see http://search.cpan.org/~neilb/Acme-Experiment-NEILB-0.01/ which contains two modules:

  • Acme::Experiment::NEILB, for which I have permissions
  • Acme::Experiment::OALDERS, for which OALDERS had permission, but I didn't.

If you look at the equivalent dist page on MetaCPAN (https://metacpan.org/release/Acme-Experiment-NEILB), there is no indication that anything is wrong. Note as well that on that on this page, it lists Acme::Experiment::OALDERS, but the link is going to Olaf's version, not mine. I understand why that is happening, but it still feels wrong.

I've raised the underlying issue (that such a "bad" dist gets indexed at all) to ANDK, with a proposal for change in behaviour. He's said this will need discussing amongst the PAUSE admins, so even if it does change, it won't be in the near future.

@ranguard
Copy link
Member

Moving to wishlist: https://github.com/CPAN-API/cpan-api/wiki/Wishlist

rwstauner added a commit that referenced this issue May 21, 2014
Until we improve the accuracy of the indexer
and are more confident in the logic.

Leave it as a feature flag so that we can toggle it on/off for testing.
Maybe we'll re-enable it someday, maybe we won't.

refs #706, #1117, #1164, #1200.
oiami pushed a commit to oiami/metacpan-web that referenced this issue Jun 9, 2014
Until we improve the accuracy of the indexer
and are more confident in the logic.

Leave it as a feature flag so that we can toggle it on/off for testing.
Maybe we'll re-enable it someday, maybe we won't.

refs metacpan#706, metacpan#1117, #1164, metacpan#1200.
oiami pushed a commit to oiami/metacpan-web that referenced this issue Jun 9, 2014
Until we improve the accuracy of the indexer
and are more confident in the logic.

Leave it as a feature flag so that we can toggle it on/off for testing.
Maybe we'll re-enable it someday, maybe we won't.

refs metacpan#706, metacpan#1117, #1164, metacpan#1200.
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

2 participants