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

#409 task 'Artifactory Tools Installer' raises error 401 #431

Merged
merged 2 commits into from
Jun 22, 2023

Conversation

Shahyads
Copy link
Contributor

@Shahyads Shahyads commented Jun 17, 2023

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • This pull request is on the dev branch.
  • I used npm run format for formatting the code before submitting the pull request.

@github-actions
Copy link

github-actions bot commented Jun 17, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Shahyads
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@@ -165,7 +165,7 @@ function createAuthHandlers(serviceConnection) {
}

// Check if Artifactory should be accessed anonymously.
if (artifactoryUser === '') {
if (artifactoryUser === '' || artifactoryUser === undefined) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if (artifactoryUser === '' || artifactoryUser === undefined) {
if (!artifactoryUser) {

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sverdlov93 sverdlov93 changed the base branch from v2 to dev June 17, 2023 09:17
@Shahyads
Copy link
Contributor Author

Shahyads commented Jun 22, 2023

thanks @sverdlov93, I just fixed it

@sverdlov93 sverdlov93 merged commit 7535cc6 into jfrog:dev Jun 22, 2023
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.

3 participants