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

Remove Uninstallation Behavior #105

Merged
merged 2 commits into from
Mar 2, 2023
Merged

Remove Uninstallation Behavior #105

merged 2 commits into from
Mar 2, 2023

Conversation

confused-Techie
Copy link
Member

Requirements

  • Filling out the template is required.

  • All new code requires tests to ensure against regressions.

    • However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.
  • Have you ran tests against this code?

  • This PR contains zero code changes.

Description of the Change

When creating the original Pulsar Backend, there was an undocumented endpoint discovered /api/:packType/:packageName/versions/:versionName/events/uninstall. This endpoint was a POST message that was triggered each time a package was uninstalled. Without much thought this was assumed to have decreased the download count of the package.

But after further investigation and communication with other Pulsar Devs we determined that this likely was not the case, as the download count for packages was unlikely to represent current installs rather it's more likely showing the total ever installs, as one would initially expect.

We then came to the conclusion that this was likely used as a way to measure current active users of any given package, and for us this seemed to much like telemetry of users.

Especially considering we had not been using this endpoint in that way, we went ahead then and decided it would be best to remove it. So this PR does just that.

Removes any logic associated with the endpoint. While the endpoint will still resolve, it will always return a successful request. This is done to prevent this change having any effect on users, and not have to bump our semver version, and not have to implement any changes on PPM in a time coordinated fashion.

So this PR should make it feel like nothing has changed, meanwhile now changing the download count of a package to be total downloads, whereas uninstalling the package does nothing.

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.

1 participant