Skip to content

Remove NuGet authentication with password option #3223

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Oct 27, 2025
Merged

Remove NuGet authentication with password option #3223

merged 9 commits into from
Oct 27, 2025

Conversation

@yordan-mitev
Copy link
Contributor

@yordan-mitev yordan-mitev commented Sep 9, 2025

  • This PR removes the option to authenticate with a username and password in the Telerik NuGet server.
  • Authentication with an API key is now the only officially supported option.

Authentication with a username and password will still work, but should not be advertised or mentioned in the docs due to security concerns.

dimodi
dimodi reviewed Sep 9, 2025
View reviewed changes
dimodi
dimodi reviewed Sep 9, 2025
View reviewed changes
dimodi
dimodi reviewed Sep 9, 2025
View reviewed changes
@yordan-mitev
Copy link
Contributor Author

yordan-mitev commented Sep 9, 2025

@dimodi , this is a quick summary of the changes since your last review:

  • Move the API key download instructions to a template.
  • Update the deployment/nuget-keys.md and installation/nuget.md files to use the instructions from the template.
  • Remove the screenshot.
  • Update the template for the Getting Started articles to reflect the requirement to use an API key instead of a password.

@yordan-mitev yordan-mitev requested a review from dimodi September 9, 2025 11:59
@yordan-mitev yordan-mitev marked this pull request as ready for review October 24, 2025 12:49
@yordan-mitev yordan-mitev requested review from a team as code owners October 24, 2025 12:49
dimodi
dimodi reviewed Oct 24, 2025
View reviewed changes
dimodi
dimodi reviewed Oct 24, 2025
View reviewed changes
dimodi
dimodi reviewed Oct 24, 2025
View reviewed changes
dimodi
dimodi reviewed Oct 24, 2025
View reviewed changes
dimodi
dimodi reviewed Oct 24, 2025
View reviewed changes
@yordan-mitev yordan-mitev requested a review from dimodi October 27, 2025 08:12
ikoevska
ikoevska approved these changes Oct 27, 2025
View reviewed changes
@dimodi dimodi added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Oct 27, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

Hello @dimodi,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

@dimodi dimodi merged commit 5893717 into master Oct 27, 2025
1 check passed
@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

Here you are the link to your PR to production: #3306

@github-actions github-actions bot mentioned this pull request Oct 27, 2025
Merged
@github-actions github-actions bot deleted the use-nuget-api-key-as-primary-auth branch October 27, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dimodi dimodi dimodi approved these changes

@ikoevska ikoevska ikoevska approved these changes

Assignees

No one assigned

Labels

merge-to-production Use this label to get a comment to choose whether to merge the PR to production

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yordan-mitev @dimodi @ikoevska